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

Full user extractor #32

Open
szattila98 opened this issue Jun 29, 2022 · 0 comments
Open

Full user extractor #32

szattila98 opened this issue Jun 29, 2022 · 0 comments
Labels
server Server side task

Comments

@szattila98
Copy link
Owner

szattila98 commented Jun 29, 2022

Implement an extractor which instead of only getting the userId from the cookie of the request, should also query the whole user from the db.
This may not be an easy task, as some kind of caching may be needed so not every request makes a database query, where it is used (but maybe not, this needs more discussion).
This would be nice because sometimes the whole user is needed and it would spare us the code duplication.
AuthenticatedUser extractor should be renamed then to AuthenticatedUserId and this should be the AuthenticatedUser. Both should remain as both will probably be needed.

@szattila98 szattila98 added server Server side task enhancement New feature or request labels Jun 29, 2022
@szattila98 szattila98 removed the enhancement New feature or request label Oct 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
server Server side task
Projects
None yet
Development

No branches or pull requests

1 participant