Skip to content

DefaultFilterChainValidator should check AuthorizationFilter #11327

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 Jun 2, 2022 · 0 comments
Closed
Tracked by #9290

DefaultFilterChainValidator should check AuthorizationFilter #11327

jzheaux opened this issue Jun 2, 2022 · 0 comments
Assignees
@jzheaux
Labels
in: web An issue in web modules (web, webmvc) status: backported An issue that has been backported to maintenance branches type: enhancement A general enhancement
Milestone
5.8.x

Comments

@jzheaux
Copy link
Contributor

jzheaux commented Jun 2, 2022

DefaultFilterChainValidator should check for duplicate AuthorizationFilter instances as well as the common error of having a login page URL protected by the AuthorizationFilter.
@jzheaux jzheaux added in: web An issue in web modules (web, webmvc) type: enhancement A general enhancement labels Jun 2, 2022
@jzheaux jzheaux added this to the 5.8.x milestone Jun 2, 2022
@jzheaux jzheaux self-assigned this Jun 2, 2022
@jzheaux jzheaux mentioned this issue Jun 7, 2022
Open
16 tasks
jzheaux added a commit that referenced this issue Jul 7, 2022
@jzheaux
Clarify variable names
cdafa4e 
Issue gh-11327
jzheaux added a commit that referenced this issue Jul 7, 2022
@jzheaux
Add AuthorizationFilter to filter chain validator
d27d431 
Closes gh-11327
@github-actions github-actions bot added the status: backported An issue that has been backported to maintenance branches label Jul 7, 2022
@github-actions github-actions bot mentioned this issue Jul 7, 2022
Closed
jzheaux added a commit that referenced this issue Jul 7, 2022
@jzheaux
Clarify variable names
ec8c133 
Issue gh-11327
@jzheaux jzheaux closed this as completed in 01ffc93 Jul 7, 2022
jzheaux added a commit that referenced this issue Jul 7, 2022
@jzheaux
Add Configuration Test
e8a7b65 
Issue gh-11327
jzheaux added a commit that referenced this issue Jul 7, 2022
@jzheaux
Add Configuration Test
c9a3d21 
Issue gh-11327
rwinch added a commit that referenced this issue Jul 8, 2022
@rwinch
Fix Formatting
d2d5313 
Issue gh-11327
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jzheaux jzheaux

Labels
in: web An issue in web modules (web, webmvc) status: backported An issue that has been backported to maintenance branches type: enhancement A general enhancement
Projects
None yet
Milestone
5.8.x
Development

No branches or pull requests
1 participant
@jzheaux