Skip to content

Commit

Permalink
Polish Documentation
Browse files Browse the repository at this point in the history
Changed indentation on saml2Login() snippets to align more closely
with surrounding documentation.

Also removed call to super.configure as this would enable formLogin as
well as httpBasic. Replaced with default endpoint authorization
statement.

Issue gh-7654
  • Loading branch information
jzheaux committed Dec 18, 2019
1 parent c745889 commit 40d4dce
Showing 1 changed file with 12 additions and 6 deletions.
Expand Up @@ -258,10 +258,12 @@ public class SecurityConfig extends WebSecurityConfigurerAdapter {
};
http
.authorizeRequests()
.anyRequest().authenticated()
.and()
.saml2Login()
.addObjectPostProcessor(processor)
.addObjectPostProcessor(processor)
;
super.configure(http);
}
}
----
Expand All @@ -282,10 +284,12 @@ public class SecurityConfig extends WebSecurityConfigurerAdapter {
authProvider.setAuthoritiesMapper(AUTHORITIES_MAPPER);
authProvider.setAuthoritiesExtractor(AUTHORITIES_EXTRACTOR);
http
.authorizeRequests()
.anyRequest().authenticated()
.and()
.saml2Login()
.authenticationManager(new ProviderManager(asList(authProvider)))
.authenticationManager(new ProviderManager(asList(authProvider)))
;
super.configure(http);
}
}
----
Expand All @@ -305,10 +309,12 @@ public class SecurityConfig extends WebSecurityConfigurerAdapter {
protected void configure(HttpSecurity http) throws Exception {
AuthenticationManager authenticationManager = new MySaml2AuthenticationManager(...);
http
.authorizeRequests()
.anyRequest().authenticated()
.and()
.saml2Login()
.authenticationManager(authenticationManager)
.authenticationManager(authenticationManager)
;
super.configure(http);
}
}
----
Expand Down

0 comments on commit 40d4dce

Please sign in to comment.