NullPointerException for Springboot @Autowired component in interceptor

Question

I have a component that's the main job is to return a Jedis instance, and it looks like the following:

@Component
public class JedisConfig {

    private Jedis jedis;

    public JedisConfig() {
        jedis = new Jedis("localhost", 6379);
    }

    public Jedis getJedis() {return jedis;}
}

I then use the Jedis instance to do some stuff in my interceptor's preHandler:

public class AuthInterceptor implements HandlerInterceptor {

    @Autowired
    private JedisConfig jc;

    public boolean preHandle(HttpServletRequest request, HttpServletResponse response, Object handler) throws Exception {
        Jedis jedis = jc.getJedis();
        
        // do some other stuff with jedis below
    }
}

But I get a NullPointerException when calling jc.getJedis() and I don't understand why it's happening.

On a related note, I did pretty much the exact same thing in a unit test and it runs fine:

@Autowired
private JedisConfig jc;

@Test
public void testJedis(){
    Jedis jedis = jc.getJedis();

    jedis.set("user", "role");
    assertThat(jedis.get("user"),is("role"));
}

Here's How I added the interceptor to the registry:

@Configuration
public class WebConfig implements WebMvcConfigurer {

    @Override
    public void addInterceptors(InterceptorRegistry registry) {
        registry.addInterceptor(new AuthInterceptor())
                .addPathPatterns("/user/roleChange");
    }
}

If anyone knows how to fix this please let me know. Thank you so much!

Is `AuthInterceptor` in the Spring context? There is no annotation on the class here so we would need to know how it is created — grekier, Jun 17 '22 at 07:55
@NikolasCharalambidis, I edited my question. Basically I just added the interceptor to a registry. — Nathan, Jun 17 '22 at 07:58

score 3 · Accepted Answer · answered Jun 17 '22 at 08:03

The AuthInterceptor is not a Spring bean, and the JedisConfig is not autowired and is null. As long as the interceptor is used in the @Configuration class, you can perform autowiring there and encapsulating the JedisConfig interceptor.

@RequiredArgsConstructor // either use Lombok or write the constructor by yourself
public class AuthInterceptor implements HandlerInterceptor {

    private final JedisConfig jc;
}

@Configuration
public class WebConfig implements WebMvcConfigurer {

    @Autowired
    private JedisConfig jc;

    @Override
    public void addInterceptors(InterceptorRegistry registry) {
        registry.addInterceptor(new AuthInterceptor(jc))
                .addPathPatterns("/user/roleChange");
    }
}

Alternatively, you can treat the AuthInterceptor as a component and autowire it in the WebConfig class:

@Component
public class AuthInterceptor implements HandlerInterceptor {

    @Autowired
    private JedisConfig jc;

    public boolean preHandle(HttpServletRequest request, HttpServletResponse response, Object handler) throws Exception {
        Jedis jedis = jc.getJedis();
        
        ...
    }
}

@Configuration
public class WebConfig implements WebMvcConfigurer {

    @Autowire
    private AuthInterceptor authInterceptor;

    @Override
    public void addInterceptors(InterceptorRegistry registry) {
        registry.addInterceptor(authInterceptor)
                .addPathPatterns("/user/roleChange");
    }
}

Thanks man! I fixed it by making Authinterceptor a bean. Btw, what's the difference between `@Component` and `@RequiredArgsConstructor`? I just did `@Component` and it seems to be working. — Nathan, Jun 17 '22 at 08:08
`@RequiredArgsConstructor` is just a Lombok thing to generate a constructor with all required fields to be initialized. — Nikolas Charalambidis, Jun 17 '22 at 08:09

NullPointerException for Springboot @Autowired component in interceptor

1 Answers1