Skip to content

Add RoleHierarchyAuthorizationManager #11304

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
Tracked by #9290
jzheaux opened this issue May 31, 2022 · 2 comments
Closed
Tracked by #9290

Add RoleHierarchyAuthorizationManager #11304

jzheaux opened this issue May 31, 2022 · 2 comments
Assignees
@evgeniycheban
Labels
in: core An issue in spring-security-core status: ideal-for-contribution An issue that we actively are looking for someone to help us with type: enhancement A general enhancement
Milestone
5.8.0-M1

Comments

@jzheaux
Copy link
Contributor

jzheaux commented May 31, 2022

To replace RoleHierarchyVoter
@jzheaux jzheaux added in: core An issue in spring-security-core type: enhancement A general enhancement status: ideal-for-contribution An issue that we actively are looking for someone to help us with labels May 31, 2022
@jzheaux jzheaux added this to the 5.8.x milestone May 31, 2022
@jzheaux jzheaux mentioned this issue May 31, 2022
Open
16 tasks
@evgeniycheban
Copy link
Contributor

I'd like to take this.

@evgeniycheban
Copy link
Contributor

@jzheaux I think instead of adding a new AuthorizationManager we can add RoleHierarchy field to AuthorityAuthorizationManager that defaults to NullRoleHierarchy along with setter method to override.

evgeniycheban added a commit to evgeniycheban/spring-security that referenced this issue May 31, 2022
@evgeniycheban
Add RoleHierarchy to AuthorityAuthorizationManager
9147567 
Added roleHierarchy field to AuthorityAuthorizationManager
that defaults to NullRoleHierarchy along with setter method to override.

Closes spring-projectsgh-11304
@evgeniycheban evgeniycheban mentioned this issue May 31, 2022
Merged
jzheaux pushed a commit that referenced this issue Jun 1, 2022
@evgeniycheban @jzheaux
Add RoleHierarchy to AuthorityAuthorizationManager
d557d2d 
Added roleHierarchy field to AuthorityAuthorizationManager
that defaults to NullRoleHierarchy along with setter method to override.

Closes gh-11304
@jzheaux jzheaux closed this as completed in 5eadcba Jun 1, 2022
@jzheaux jzheaux modified the milestones: 5.8.x, 5.8.0-M1 Jun 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@evgeniycheban evgeniycheban

Labels
in: core An issue in spring-security-core status: ideal-for-contribution An issue that we actively are looking for someone to help us with type: enhancement A general enhancement
Projects
None yet
Milestone
5.8.0-M1
Development

No branches or pull requests
2 participants
@jzheaux @evgeniycheban