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 jakarta security annotation mapping #775
Add jakarta security annotation mapping #775
Conversation
… to NamedAnnotationMapper
…ir equivalent javax NamedAnnotationMapper
822cf7d
to
6c987a4
Compare
Thanks code changes look good, we will need a test though. Probably just the same as https://github.com/micronaut-projects/micronaut-security/blob/master/security/src/test/groovy/io/micronaut/security/rolesallowed/RolesAllowedSpec.groovy with different annotations |
…cs and replace test controllers with jakarta annotated controllers
@graemerocher I've extended the original specs and replaced the controllers so that they use the jakarta annotation. |
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.
@frehov Please add the annotations to the documentation as well. See secured.adoc
...notations/src/main/java/io/micronaut/security/annotation/JakartaDenyAllAnnotationMapper.java
Show resolved
Hide resolved
...tations/src/main/java/io/micronaut/security/annotation/JakartaPermitAllAnnotationMapper.java
Show resolved
Hide resolved
...ions/src/main/java/io/micronaut/security/annotation/JakartaRolesAllowedAnnotationMapper.java
Show resolved
Hide resolved
@frehov Thanks for the contribution! |
Switched over the AnnotationMappers for javax security annotations to NamedAnnotationMapper as discussed in #774.
Added the Jakarta equaivalent mappers by extending the javax ones and overriding the namespace.
closes #774