-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Add a JPA Identity Extension #4347
Comments
Looking at it for prototyping. |
Well, I stopped looking at it, but I do have a branch with the start of this. I hit the issue that it appears that the credentials are being looked up outside of a scope in which ArC can deliver an |
We need to create the request scope earlier then I guess. I was trying to avoid this as there is some complexity in dealing with this if the thread is propagated between different threads. Creating an EntityManager is also a solution (and should not really be a perf issue, they are supposed to be fairly cheap), but it means that if we are going to stick the user entity into the SecurityIdentity then it will be a detached entity, although this might be ok. |
yes +1 to detach entities as long as there's no uninitialized paths in the loaded graph - I assume it's going to be a simple, relationless entity? |
It's almost done here: #5660 |
This extension would allow for an easy annotation driven approach to creating JPA entities that represent a user. An entity can be annotated with @Username and @password (and maybe @salt) to inform Quarkus that this entity represents a user. Quarkus will then automatically wire up authentication based on this entity.
The text was updated successfully, but these errors were encountered: