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

doc: fix security-jwt.adoc according to quickstart project #6708

Merged
merged 1 commit into from
Jan 22, 2020

Conversation

kdnakt
Copy link
Contributor

@kdnakt kdnakt commented Jan 21, 2020

Optional<String> caused javax.enterprise.inject.UnsatisfiedResolutionException: Unsatisfied dependency for type java.util.Optional<java.lang.String> and qualifiers [@Claim(standard = birthdate)] when starting the app in dev mode.

Also I fixed some code in the document according to quickstart project implementation:
https://github.com/quarkusio/quarkus-quickstarts/blob/master/security-jwt-quickstart/src/main/java/org/acme/jwt

@gastaldi
Copy link
Contributor

Nice. Can you squash all your commits into a single one?

@gsmet
Copy link
Member

gsmet commented Jan 22, 2020

I squashed to one commit so that it's easier to backport.

Merging, thanks!

@gsmet gsmet merged commit b53125d into quarkusio:master Jan 22, 2020
@gsmet gsmet added this to the 1.3.0 milestone Jan 22, 2020
@kdnakt
Copy link
Contributor Author

kdnakt commented Jan 22, 2020

Thank you for the squash! I will squash logs next time!!

@gsmet gsmet modified the milestones: 1.3.0, 1.2.0.Final Jan 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants