Skip to content

Refactor some of the email auth flow stuff for better separation of concerns #109

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

Open
chriscarrollsmith opened this issue Mar 15, 2025 · 0 comments

Comments

@chriscarrollsmith
Copy link
Contributor

The more traditional CRUD app approach to stuff like email reset tokens, especially given that the tokens have their own database models, would likely be to move the associated routes out of routers/account.py and into their own routers that are specific to the tokens, treating the tokens as their own resources to be gotten and posted with resource-specific GET and POST routes. We've also got a fair amount of logic aggregated in utils/auth.py, including regex helpers, password validation helpers, email flow helpers, etc., and maybe we should be moving these to separate utility files.

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

1 participant