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

Setting for requiring email/passwords on user accounts? #1151

Open
notecardPasta opened this issue Dec 5, 2024 · 1 comment
Open

Setting for requiring email/passwords on user accounts? #1151

notecardPasta opened this issue Dec 5, 2024 · 1 comment

Comments

@notecardPasta
Copy link

Accounts that register with a provider (such as TH or DA) skip needing an email and password, but this would lock also them out of a lorekeeper site should that provider become unavailable. Should there be a config setting or something to enable a check for if an account has an email and if not redirect them to settings to add one?

This does essentially nullify the one-step-signup of alias registration in general when the setting's on, but it could be a useful thing to have available if a LK either decides against using a certain site for authentication or if a site gives a headsup it'll be closing down soon (or is getting buggier...looking at a certain green art site) so that people can gradually be nudged into creating backup credentials

@SpeedyD
Copy link
Contributor

SpeedyD commented Dec 5, 2024

I think it'd be a good idea to enforce email and password being added to an account. Auth is good for authentication, but it is up to the whims of the specific site whether or not it remains good for logging in. (Looking at you, Muskrat, making API use cost money suddenly..)

Having email and password REQUIRED BY DEFAULT is just a good backup, we've had plenty of cases where users couldn't get in.

I strongly suggest having this be forced even retroactive, similar to requiring to auth or requiring to add a birthday.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants