Skip to content

DelegatingAuthorizationManager Should Fire Events #9288

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
Closed
jzheaux opened this issue Dec 16, 2020 · 3 comments
Closed

DelegatingAuthorizationManager Should Fire Events #9288

jzheaux opened this issue Dec 16, 2020 · 3 comments
Assignees
@parikshitdutta @jzheaux
Labels
in: web An issue in web modules (web, webmvc) type: enhancement A general enhancement
Milestone
5.7.0-RC1

Comments

@jzheaux
Copy link
Contributor

jzheaux commented Dec 16, 2020

Firing success and failure events from DelegatingAuthorizationManager would simplify auditing. AuthorizationSuccessEvent and AuthorizationFailureEvent should be suitable.
@jzheaux jzheaux added in: web An issue in web modules (web, webmvc) type: enhancement A general enhancement status: ideal-for-contribution An issue that we actively are looking for someone to help us with labels Dec 16, 2020
This was referenced Dec 16, 2020
Merged
Open
@parikshitdutta
Copy link
Contributor

Hi @jzheaux,
Please assign me the task, I can take it.

Thank you.

@jzheaux
Copy link
Contributor Author

jzheaux commented Mar 29, 2021

Thanks, @parikshitdutta, it's yours!

I think you may want to consider #9286 as well.

@jzheaux jzheaux removed the status: ideal-for-contribution An issue that we actively are looking for someone to help us with label Mar 29, 2021
This was referenced Mar 29, 2021
Closed
Closed
@parikshitdutta
Copy link
Contributor

Hi @jzheaux, Please have a look at my submitted PR #9527, I will look forward to your feedback.

Thank you.

jzheaux added a commit that referenced this issue Mar 29, 2022
@jzheaux
Simplify PrePostMethodSecurityConfiguration
a43677d 
Issue gh-9288
jzheaux pushed a commit that referenced this issue Mar 29, 2022
@parikshitdutta @jzheaux
Add authorization events
bd94348 
Closes gh-9288
jzheaux added a commit that referenced this issue Mar 29, 2022
@jzheaux
Polish Authorization Event Support
061f69e 
- Added spring-security-config support
- Renamed classes
- Changed contracts to include the authenticated user and secured
object
- Added method security support

Issue gh-9288
jzheaux added a commit that referenced this issue Mar 29, 2022
@jzheaux
Document Authorization Events
04c4833 
Issue gh-9288
jzheaux added a commit that referenced this issue Mar 29, 2022
@jzheaux
Simplify PrePostMethodSecurityConfiguration
fa574c8 
Issue gh-9288
jzheaux added a commit that referenced this issue Mar 29, 2022
@jzheaux
Polish Authorization Event Support
bdd5f86 
- Added spring-security-config support
- Renamed classes
- Changed contracts to include the authenticated user and secured
object
- Added method security support

Issue gh-9288
jzheaux added a commit that referenced this issue Mar 29, 2022
@jzheaux
Document Authorization Events
ce720ad 
Issue gh-9288
@jzheaux jzheaux added this to the 5.7.0-RC1 milestone Mar 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@parikshitdutta parikshitdutta

@jzheaux jzheaux

Labels
in: web An issue in web modules (web, webmvc) type: enhancement A general enhancement
Projects
None yet
Milestone
5.7.0-RC1
2 participants
@parikshitdutta @jzheaux