Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Don't require e-mail verification after logging in for the first time #185

Open
luketeam5 opened this issue Jul 22, 2024 · 2 comments
Open
Labels
enhancement New feature or request Fixed in Dev This issue has been fixed in dev

Comments

@luketeam5
Copy link

What feature do you want to see added?

Hi,

so there's a little issue when using full e-mail registration. As of now, after logging in for the first time with the temporary password (that's sent via e-mail) I want my players to be able to change their password . However, when they attempt /changepassword they get asked for a verification, which in my mind is quite stupid, given that they just proved it is them when they logged in for the FIRST TIME a few seconds ago!

I think it would be great to have either some sort of a time limit, where logging in for the first time let's you perform the dangerous commands OR if if the plugin could count password resets as verification.

Are there any alternatives?

No, only way is to let players pick their password on account registration, however I want to verify the e-mail first.

Anything else?

No response

@luketeam5 luketeam5 added the enhancement New feature or request label Jul 22, 2024
@HaHaWTH
Copy link
Owner

HaHaWTH commented Jul 22, 2024

I added a configuration for changepassword command, you can disable verification to meet your needs. XD
btw it is safe don't worry
commit: bd7a255

@luketeam5
Copy link
Author

I added a configuration for changepassword command, you can disable verification to meet your needs. XD btw it is safe don't worry commit: bd7a255

Wow, this was fast, given that I did not even receive any response on the AuthmeReloaded feature request after more than a month, you're amazing.

I mean, it's not the perfect solution (the temporary immunity after registering would be better, so I'll keep this open) but it will do for now (and probably turn to an "permanent temporary solution").

Thanks!

@HaHaWTH HaHaWTH added the Fixed in Dev This issue has been fixed in dev label Jul 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Fixed in Dev This issue has been fixed in dev
Projects
None yet
Development

No branches or pull requests

2 participants