StackOverflowError in spring oauth2 with custom ClientDetailsService

后端 未结 4 451
面向向阳花
面向向阳花 2021-02-04 11:02

I made my own implementation of ClientDetailsService:

@Service
public class JpaClientDetailsService implements ClientDetailsService {
    @Autowired
    private          


        
相关标签:
4条回答
  • 2021-02-04 11:11

    What ended up working for me was adding the ClientDetailsService @Bean to the @EnableAuthorizationServer class:

    @Configuration @EnableAuthorizationServer public class AuthorizationServerConfiguration implements AuthorizationServerConfigurer { ... @Autowired ClientDetailsService clientDetailsService; ... @Bean public ClientDetailsService clientDetailsService() { return new CustomClientDetailsServiceImpl(); } ... }

    0 讨论(0)
  • 2021-02-04 11:22

    I do not understand why, but if I inject my bean directly instead of injecting the interface, it works :

    public class OAuth2ServerConfig extends AuthorizationServerConfigurerAdapter {
    ...
      @Autowired
      private JpaClientDetailsService clientDetailsService;
    ...
    

    it also works if I annotate my service with @Primary annotation:

    @Service
    @Primary
    public class JpaClientDetailsService implements ClientDetailsService {
    
    0 讨论(0)
  • 2021-02-04 11:32

    I would say that the best solution is to be explicit - if you are autowiring a clientDetailsService - then say so.

    @Autowired
    ClientDetailsService myClientDetailsService;
    
    @Override
    public void configure(final AuthorizationServerEndpointsConfigurer endpoints) throws Exception {
    
            endpoints.setClientDetailsService(myClientDetailsService);
           ....
    }
    

    However, My problem was slightly different and the above solutions did not work. Here are the conditions that created it. I created a CustomTokenEndpointAuthenticationFilter - which required an instance of OAuth2RequestFactory. I created my instance of OAuth2RequestFactory with a call to endpoints.getOAuth2RequestFactory(); before the clientDetailsService had been set.

    If The OAuth2RequestFactory gets created this way, DefaultOAuth2RequestFactory gets created with a default clientDetailsService, so even if you set the clientDetailsService later explicitly in the AuthorizationServerEndpointsConfigurer it will not be in the OAuth2RequestFactory used by your custom Filter.

    So in sum in this edge case

    public void configure(final AuthorizationServerEndpointsConfigurer endpoints) throws Exception {
         oAuth2RequestFactory = endpoints.getOAuth2RequestFactory();
         endpoints.setClientDetailsService(myClientDetailsService);
        ...
        }
    

    wont work but

    public void configure(final AuthorizationServerEndpointsConfigurer endpoints) throws Exception {
    endpoints.setClientDetailsService(myClientDetailsService);
     oAuth2RequestFactory = endpoints.getOAuth2RequestFactory();
    ...
    }
    

    will.

    Another way to be sure is to create your own OAuth2RequestFatory

      oauth2RequestFactory = new DefaultTokenRequestFactory(myClientDetailsService);
    
    0 讨论(0)
  • 2021-02-04 11:33

    I had similar problem. Finally I resolved bug when I gave my clientDetailsService another name i.e. myClientDetailsService and then injected this by name in AuthorizationServerConfig class:

    @Configuration
    @EnableAuthorizationServer
    public class AuthorizationServerConfig extends AuthorizationServerConfigurerAdapter {
    
        @Resource(name = "myClientDetailsService")
        private ClientDetailsService clientDetailsService;
    ...
    

    I think that if my own implementation of ClientDetailsService wasn't yet created Spring inject into AuthorizationServerConfig some kind of proxy.

    So, if you want to resolve this kind of bug you must be sure that Spring inject proper ClientDetailsService in AuthorizationServerConfig. You can achieve this if you:

    1. give spring information about preference of your own ClientDetailsService (Arnaud answer), or
    2. inject this service by name
    0 讨论(0)
提交回复
热议问题