How to define order of spring security filter chain

Question

How to define order of spring security filter chain

Closed this issue 7 years ago · 20 comments

kopax commented 8 years ago

How to define order of spring security filter chain

I am using the following dependency :

spring-boot 1.5.2.RELEASE
- spring-boot-starter-security
spring-security-oauth2 2.1.0.RELEASE

I have configured two springSecurityFilterChain using:

(1) WebSecurityConfigurerAdapter as first security layer (Basic auth)
(2) ResourceServerConfigurerAdapter as second security layer. (Bearer auth)

My endpoint should be secured like this :

/login (anonymous)
/ping (anonymous)
/oauth/** : filter (1)
/**: filters (1, 2)

(1) look like this:

@Configuration
@EnableWebSecurity
@EnableGlobalMethodSecurity(prePostEnabled = true)
@Order(2)
public class SecurityConfig extends WebSecurityConfigurerAdapter {

	@Override
	protected void configure(HttpSecurity http) throws Exception {
		http
			.sessionManagement()
				.sessionCreationPolicy(SessionCreationPolicy.NEVER)
				.and()
			.csrf().disable()
			.authorizeRequests()
				.antMatchers(HttpMethod.GET, HttpPathStore.PING).permitAll()
				.antMatchers("/login").permitAll()
				.antMatchers("/logout").hasRole("USER")
				.antMatchers("/**").hasRole("USER")
				.anyRequest().authenticated()
			.and()
				.exceptionHandling()
				.accessDeniedHandler(accessDeniedHandler)
				.authenticationEntryPoint(authenticationEntryPoint)
			.and()
				.formLogin()
				.loginProcessingUrl(HttpPathStore.LOGIN)
				.successHandler(authenticationSuccessHandler)
				.failureHandler(authenticationFailureHandler)
				.permitAll()
			.and()
				.logout()
				.logoutUrl(HttpPathStore.LOGOUT)
				.logoutSuccessUrl(HttpPathStore.LOGIN_FROM_LOGOUT)
				.logoutSuccessHandler(logoutSuccessHandler)
				.permitAll();
	}
	...
}

(2) look like this:

@Configuration
@EnableResourceServer
@Order(3)
public class ResourceServerConfig extends ResourceServerConfigurerAdapter {

	@Override
	public void configure(HttpSecurity http) throws Exception {
		http
			.authorizeRequests()
				.antMatchers(HttpMethod.GET, HttpPathStore.PING).permitAll()
				.antMatchers("/login").permitAll()
				.antMatchers("/logout").permitAll()
				.antMatchers(
					"/**"
				).access("#oauth2.hasScope('write') " +
							"and #oauth2.clientHasRole('ROLE_CLIENT') " +
							"and hasRole('USER')");
	}

	...
}

Problem

If I set @Order on:

(1) : @Order(SecurityProperties.ACCESS_OVERRIDE_ORDER)
(2) : [DEFAULT]

Expected: as a user with a session and without a jwt, I expect to see the the response 401 from (2)

Result: as a user with a session and without a jwt, I have the response 401 from (1).

If I set @Order on:

(1) : @Order(2)
(2) : @Order(3)

Expected: Order the web securityFilterChain before OAuth securityFilterhChain

Result: OAuth securityFilterChain configuration seems to be ignored by spring.

Is there a way to get my configuration to have OAuth security after Basic security ?

Answer 1 · 2017-03-17T22:49:47.000Z

Is related with https://github.com/spring-projects/spring-boot/wiki/Spring-Boot-1.5-Release-Notes#oauth-2-resource-filter ?

Answer 2 · 2017-03-17T23:30:36.000Z

Is related with https://github.com/spring-projects/spring-boot/wiki/Spring-Boot-1.5-Release-Notes#oauth-2-resource-filter ?

Yes it is on the same topic. I can't tell you if my issue is due to these changes ?

Do you wan't me to redo the test on spring-boot 1.4.1.RELEASE ?

Answer 3 · 2017-03-19T13:43:44.000Z

@kazuki43zoo according to your link and the setup I want. What @order value should I use for WebSecurityConfigurerAdapter and ResourceServerConfigurerAdapter ?

Answer 4 · 2017-03-19T16:41:50.000Z

Could you try as follow ?

@Configuration
@EnableResourceServer
public class ResourceServerConfig extends ResourceServerConfigurerAdapter {

	@Override
	public void configure(HttpSecurity http) throws Exception {
		http.antMatcher("/oauth/**") // ### Specify path pattern that need OAuth authentication(Bearer auth) and authorization
			.authorizeRequests()
				.antMatchers(
					"/**"
				).access("#oauth2.hasScope('write') " +
							"and #oauth2.clientHasRole('ROLE_CLIENT') " +
							"and hasRole('USER')");
	}
	...
}

Answer 5 · 2017-03-19T18:38:44.000Z

@kazuki43zoo I would like to know if there is a difference between this

http.antMatcher("/oauth/**") // ### Specify path pattern that need OAuth authentication(Bearer auth) and authorization
		.authorizeRequests()
			.antMatchers(
				"/**"
			).access("#oauth2.hasScope('write') " +
						"and #oauth2.clientHasRole('ROLE_CLIENT') " +
						"and hasRole('USER')");

and this

http
		.authorizeRequests()
			.antMatchers(
                                "/oauth/**", // ### Specify path pattern that need OAuth authentication(Bearer auth) and authorization
				"/**"
			).access("#oauth2.hasScope('write') " +
						"and #oauth2.clientHasRole('ROLE_CLIENT') " +
						"and hasRole('USER')");

Could you try as follow ?

With what @Order exactly?

I did try with @Order(2), no @Order annotation on ResourceServerConfig.

Same thing as before, having @Order(2) on WebMvcSecurity result in an ignored OAuth2 security filter chain.

Answer 6 · 2017-03-20T01:59:41.000Z

In my understand ...

Default orders are as follow on Spring Boot:

A security filter chain that configure by a extends class of WebSecurityConfigurerAdapter -> 100
A resource server filter chain that configure by Spring Boot -> 2147483639 (= SecurityProperties.ACCESS_OVERRIDE_ORDER - 1)

In other words, a security filter chain that configure by a extends class of WebSecurityConfigurerAdapter will apply by default. (resource server filter chain will be not used)

If you want to apply a resource server filter chain for /oauth/***, you need to specify order as follow:

(1) : ACCESS_OVERRIDE_ORDER (= 2147483640)
(2) : [DEFAULT] (= 2147483639)

or

(1) : [DEFAULT] (= 100)
(2) : 99 or under (e.g: security.oauth2.resource.filter-order=99)

And specify a matcher pattern on (2) as follow:

http.antMatcher("/oauth/**")
    .authorizeRequests()
        ....

As result...

You can use a resource server filter chain for request that matches /oauth/** and you can use a security filter chain that configure by a extends class of WebSecurityConfigurerAdapter for other request (e.g: /login, /ping and more ... ).

If you need more information, i think you should be provide a reproduce project that can be run on GitHub.
And i think your question should be posted to the StackOverflow.

Thanks.

Answer 7 · 2017-03-20T10:44:09.000Z

Thanks for the quick reply.

It seems from the javadoc that http.antMatcher("/oauth/**") is the same as http.requestMatchers().antMatchers("/oauth/**)

Allows configuring the {@link HttpSecurity} to only be invoked when matching the
provided ant pattern. If more advanced configuration is necessary, consider using
{@link #requestMatchers()} or {@link #requestMatcher(RequestMatcher)}.

I have tried both of the configuration. They seems to be able to protect the endpoint /oauth/**, but all the endpoints /** should be OAuth secure.

I have created a demo repository.

There's a README with a sum up of the security strategy.

Reproduction

Clone the project

 git clone https://github.com/kopax/spring-security-oauth-issues-1024 && cd spring-security-oauth-issues-1024

start the server

 ./gradlew build --info && java -jar build/libs/api-oauth2.jar --spring.profiles.active=default

get a cookie

 export COOKIE=$(curl -v --silent http://localhost:8081/ 2>&1 | grep cookie -i | awk -F ' ' '{print $3}')

authenticate the cookie

 export COOKIE=$(curl --cookie "$COOKIE" -d username=admin -d password=verysecret -v --silent http://localhost:8081/login 2>&1 | grep cookie -i | awk -F ' ' '{print $3}')

try to get a secured oauth resource at /

 curl --cookie "$COOKIE" -v http://localhost:8081/

Expected

http status code 401 due to missing header Authorization

Result

http status code 200

Useful information:

Spring server:

http://localhost:8081/

Security account (cookie):

username: admin
password: verysecret

OAuth account (jwt):

client_id: myfirstapp
client_secret: test
redirect_uri: http://localhost:8081/
access_token_uri: http://localhost:8081/oauth/token
authorization_uri: http://localhost:8081/oauth/authorize
authorization_grant: code
redirect_uri: http://localhost:8081/cb/myfirstapp

Answer 8 · 2017-03-20T14:55:56.000Z

Hi, It is still possible to access without a Bearer. See my reply on PR

Every configuration I have tried just can't work together on the same endpoint.
As soon a route is added to (1), it get canceled in (2).

Are you absolutely sure that spring-security and spring-security-oauth can work on the same endpoint ?

It's also possible that the API must have separate endpoints.
(I haven't found any project that use both plugin on top of each for a defined endpoint.)

Answer 9 · 2017-05-04T19:46:25.000Z

I am having the same problem. I want ldap authentication for some resources and oauth on others and can only get one or the other to work. Not both. Have you made any progress or do you have a work around?

Answer 10 · 2017-05-04T21:03:41.000Z

It is one or the other. You can't have both.

Answer 11 · 2017-10-26T01:43:25.000Z

I have the same problem

Answer 12 · 2018-03-17T14:08:29.000Z

Could you try following code. It works fine for me.

https://spring.io/guides/topicals/spring-security-architecture/
"The most important feature of this dispatch process is that only one chain ever handles a request."

@EnableResourceServer
@Configuration
public class ResourceServerConfiguration extends ResourceServerConfigurerAdapter {

    @Override
    public void configure(ResourceServerSecurityConfigurer resources) throws Exception {
        super.configure(resources);
    }

    @Override
    public void configure(HttpSecurity http) throws Exception {
        http.requestMatchers()
                // For org.springframework.security.web.SecurityFilterChain.matches(HttpServletRequest)
                .requestMatchers(
                        new NegatedRequestMatcher(
                                new OrRequestMatcher(
                                        new AntPathRequestMatcher("/login"),
                                        new AntPathRequestMatcher("/logout"),
                                        new AntPathRequestMatcher("/oauth/authorize"),
                                        new AntPathRequestMatcher("/oauth/confirm_access")
                                )
                        )
                )
                .and()
                .authorizeRequests().anyRequest().authenticated();
    }
}

@Configuration
public class WebSecurityConfiguration extends WebSecurityConfigurerAdapter {

    @Override
    protected void configure(HttpSecurity http) throws Exception {
        http.requestMatchers()
                // For org.springframework.security.web.SecurityFilterChain.matches(HttpServletRequest)
                .requestMatchers(
                        new OrRequestMatcher(
                                new AntPathRequestMatcher("/login"),
                                new AntPathRequestMatcher("/logout"),
                                new AntPathRequestMatcher("/oauth/authorize"),
                                new AntPathRequestMatcher("/oauth/confirm_access")
                        )
                )
                .and()
                .authorizeRequests().anyRequest().authenticated()
                .and()
                .formLogin().permitAll()
                .and()
                .logout().permitAll();
    }
}

jdk-10+46_linux-x64
Spring Boot 2.0.0.RELEASE

Answer 13 · 2018-04-14T16:06:34.000Z

@namioka Shouldn't ResourceServerConfiguration only match negatively on /login and /logout? And WebSecurityConfiguration negatively on /oauth/**?

Answer 14 · 2018-04-29T03:49:58.000Z

@lilalinux '/oauth/authorize' and '/oauth/confirm_access' require the user authentication (Not resource server authentication by access tokens).

Answer 15 · 2018-06-13T10:45:36.000Z

@kopax Did you resolve this issue? I am having same trouble when selecting HttpBasic and OAuth on the basis of Endpoint.

Answer 16 · 2018-06-22T09:53:53.000Z

Me too. With Spring Boot 1.5.8.RELEASE, Spring Security 4.2.3.RELEASE. OAuth filter chain was ignored.

Answer 17 · 2018-06-25T07:04:24.000Z

@kevinojt I have figured out a solution for this. Look at my answer
https://stackoverflow.com/questions/50796327/can-we-use-httpbasic-and-oauth2-both-for-an-api-depending-on-path-variable/50877797#50877797

Answer 18 · 2018-08-08T19:43:25.000Z

Thanks @kevinojt ! It worked perfect for me.

Answer 19 · 2018-08-10T13:51:38.000Z

@cpandya231 yes, it works.

Here is my solution same as yours.

/**
 * Config Spring security policy.
 *
 * About @Order
 * With SpringBoot must set @Order(SecurityProperties.ACCESS_OVERRIDE_ORDER)
 * on WebSecurityConfiguration or Spring Security will
 * put UsernamePasswordAuthenticationFilter before OAuth2AuthenticationProcessingFilter.
 * It means OAuth2 authentication of resource server will not work.
 *
 */
@Configuration
@EnableWebSecurity(debug = true)
@Order(SecurityProperties.ACCESS_OVERRIDE_ORDER)
public class WebSecurityConfigurer extends WebSecurityConfigurerAdapter {

  @Override
  @Bean
  public AuthenticationManager authenticationManagerBean() throws Exception {
    return super.authenticationManagerBean();
  }

  @Override
  @Bean
  public UserDetailsService userDetailsServiceBean() throws Exception {
    return super.userDetailsServiceBean();
  }
// Other code been omitted
}

Answer 20 · 2018-08-13T05:12:50.000Z

@kevinojt Please respond on stackoverflow, need to improve reputation points.