-
-
Notifications
You must be signed in to change notification settings - Fork 9
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 jwt reflect and resource config to support jwt authentication type #59
Conversation
this will be very helpful, thank you, @atomfrede!👏 |
ad29327
to
493c0d8
Compare
493c0d8
to
3cdda22
Compare
Looks like the threadpool for sending mails async is not working properly (non webflux). Going to investigate today. Besides that it looks quite well. |
Very strange, after I disabled async mail sending, the thymeleaf template can't be compile, because of an error with |
Seems I found a solution. Let me update the PR |
737cb1b
to
5e2e5cc
Compare
@mraible There seems to be an issue with macos ci. Will restart the failed builds, but everything works fine, now also with JWT 🎉 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
macOS docker install is flaky. It should be ok to merge on install failures.
Samples are ok now, should be ok to merge if succeed.
I have fixed remaining issues with webflux and reflect config. Macos build is working fine locally, so as @mshima said flaky environment. I would say let's merge it and create a new release. |
bounty claimed https://opencollective.com/generator-jhipster/expenses/91302 |
Native image can be build with jwt support and started. I got some errors when I tried to save a user (entity) but I seemed unrelated. Will open another issue for that.
@alina-yur especially for you
closes #48