Skip to content
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

Documentation : Spring security architecture guide describes old auto configuration from Spring Boot 1.x #10359

Closed
aashaysaralkar opened this issue Oct 11, 2021 · 3 comments
Labels
type: task A general task

Comments

@aashaysaralkar
Copy link

Spring security architecture guide needs to be changed to reflect the simplification done to the security auto configuration in Spring Boot 2.x. Documentation says:

A vanilla Spring Boot application with no custom security configuration has a several (call it n) filter chains, where usually n=6. The first (n-1) chains are there just to ignore static resource patterns, like /css/** and /images/**, and the error view: /error.

But SpringBootWebSecurityConfiguration no longer ignores them explicitly after the change in 2.6.0-M3 with this commit and every thing is secured by default in a vanilla setup with a single SecurityFilterChain.

@aashaysaralkar aashaysaralkar added status: waiting-for-triage An issue we've not yet triaged type: bug A general bug labels Oct 11, 2021
@sjohnr sjohnr added type: task A general task and removed status: waiting-for-triage An issue we've not yet triaged type: bug A general bug labels Oct 11, 2021
@jzheaux
Copy link
Contributor

jzheaux commented Oct 13, 2021

@aashaysaralkar I think you make some good points here. Will you please post this issue instead to https://github.com/spring-guides/top-spring-security-architecture/issues and we can take a look there?

@aashaysaralkar
Copy link
Author

@jzheaux sure! I will close the issue here.

@jzheaux
Copy link
Contributor

jzheaux commented Oct 14, 2021

Thanks, @aashaysaralkar!

For folks who land here, the issue is moved to spring-attic/top-spring-security-architecture#19

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: task A general task
Projects
None yet
Development

No branches or pull requests

3 participants