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

Revert changes to password resets / auth_tokens table #3715

Open
misaugstad opened this issue Nov 1, 2024 · 0 comments
Open

Revert changes to password resets / auth_tokens table #3715

misaugstad opened this issue Nov 1, 2024 · 0 comments
Assignees

Comments

@misaugstad
Copy link
Member

Brief description of problem/feature

As part of adding unified login (#3712), we also centralized the password resetting functionality. The initial issue I was going to bring up is that we have a process running in each city to clear out expired password reset tokens from the auth_tokens table, but since the table is shared, it's overkill to have every city doing this.

As I think further, I think that it may have been a mistake to centralize this functionality. Maybe later on I'll remember why I did it in the first place, but for I don't see any benefit to it being done this way. You get an email with a password reset link. Now you could edit the link to change which server you're going to and it would still work; it seems like that's the only "benefit", but there's no reason why anyone would do that. So maybe we should just move it back to the separate cities' schemas!

Potential solution(s)

Moving back to the separate cities' schemas should be trivially easy. We just need to remove the explicit references to the sidewalk_login schema wherever we're using the auth_tokens table. We didn't delete the city-specific ones yet, so it will default back to using those if we remove the explicit references to the new schema.

Then I'll just delete the corresponding table that was added to the sidewalk_login schema.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: next up
Development

No branches or pull requests

1 participant