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
Update to smallrye-jwt-2.4.0 #13497
Update to smallrye-jwt-2.4.0 #13497
Conversation
<dependency> | ||
<groupId>io.smallrye</groupId> | ||
<artifactId>smallrye-jwt-build</artifactId> | ||
<version>${smallrye.jwt.version}</version> |
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.
Is the version still necessary here?
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.
@gastaldi Sorry, missed this question - I don't know - it is not in Quarkus bom/application/pom.xml
dependencies management section yet as at the moment quarkus-smallrye-jwt
/quarkus-oidc
depend on it indirectly.
I can update this section later, as part of the fix for #13499, to include io.quarkus/quarkus-smallrye-jwt-build
(no version) ?
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.
I think adding it to the BOM should be done in the same commit.
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.
So you can remove the version now from here now as it will automatically come with the BOM.
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.
@gsmet, yeah, sorry, I realized it once I switched my laptop off and went for a walk :-), sorry :-)
<dependency> | ||
<groupId>io.smallrye</groupId> | ||
<artifactId>smallrye-jwt-build</artifactId> | ||
<version>${smallrye.jwt.version}</version> |
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.
I think adding it to the BOM should be done in the same commit.
aadc97e
to
204694c
Compare
204694c
to
6ae4a9c
Compare
No description provided.