Skip to content

set AuthorizationEventPublisher for Jsr250MethodInterceptor and SecuredMethodInterceptor #14416

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jan 18, 2024
Merged

set AuthorizationEventPublisher for Jsr250MethodInterceptor and SecuredMethodInterceptor #14416

merged 1 commit into from
Jan 18, 2024

Conversation

kse-music
Copy link
Contributor

Closes 14401

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Jan 8, 2024
jzheaux
jzheaux requested changes Jan 8, 2024
View reviewed changes
Copy link
Contributor

@jzheaux jzheaux left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, @kse-music! In addition to my inline feedback, please also update your commit message to include the ticket like so:

Use AuthorizationEventPublisher

Closes gh-14401

And before submitting, please run ./gradlew format on the codebase to ensure the code you add meets the formatting guidelines.

@jzheaux jzheaux self-assigned this Jan 8, 2024
@jzheaux jzheaux added in: config An issue in spring-security-config type: enhancement A general enhancement and removed status: waiting-for-triage An issue we've not yet triaged labels Jan 8, 2024
@kse-music kse-music closed this Jan 9, 2024
@kse-music kse-music deleted the fix-gh14401 branch January 9, 2024 01:13
@kse-music kse-music restored the fix-gh14401 branch January 9, 2024 01:18
@kse-music kse-music reopened this Jan 9, 2024
@kse-music
Copy link
Contributor Author

Thanks @jzheaux ,will do as you suggest.
Updated the branch.

@kse-music @jzheaux
Use AuthorizationEventPublisher Bean
d45327d 
- For Jsr250MethodInterceptor and SecuredMethodInterceptor

Closes spring-projectsgh-14401
@jzheaux jzheaux added this to the 6.3.0-M2 milestone Jan 18, 2024
@jzheaux jzheaux added the status: duplicate A duplicate of another issue label Jan 18, 2024
@jzheaux jzheaux merged commit 3f65f60 into spring-projects:main Jan 18, 2024
@kse-music kse-music deleted the fix-gh14401 branch January 19, 2024 14:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jzheaux jzheaux jzheaux approved these changes

Assignees

@jzheaux jzheaux

Labels
in: config An issue in spring-security-config status: duplicate A duplicate of another issue type: enhancement A general enhancement
Projects
None yet
Milestone
6.3.0-M2
Development

Successfully merging this pull request may close these issues.

AuthorizationEventPublisher not set for jsr250AuthorizationMethodInterceptor
3 participants
@kse-music @jzheaux @spring-projects-issues