Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Spring and @Autowired on a DelegatingFilterProxy

I'm trying to inject a spring bean into a filter, but can't make it work.

The bean injected is always "null". I succeed autowiring this same bean in Controllers and HandlerInterceptors so it's correctly annotated.

The filter class is under the same base-package of the rest of Controllers.

This is the relevant part of my web.xml

  <filter>
    <filter-name>CheckSession</filter-name>
    <filter-class>org.springframework.web.filter.DelegatingFilterProxy</filter-class>
  </filter>
  <filter-mapping>
    <filter-name>CheckSession</filter-name>
    <url-pattern>/panel/*</url-pattern>
  </filter-mapping>

This is the code for the filter

@Component 
public class CheckSession extends OncePerRequestFilter implements Filter {

    @Autowired private Usuario usuario;

    @Override
    protected void doFilterInternal(
        HttpServletRequest request,
        HttpServletResponse response, FilterChain chain)
    throws ServletException, IOException {

        //  always null
        System.out.println("autowired " + usuario);
        chain.doFilter(request,  response);
    }
}

The filter is triggering on every request.

These are the annotations in the "Usuario" bean

@Component
@Scope(value="session", proxyMode=ScopedProxyMode.TARGET_CLASS)
public class Usuario implements java.io.Serializable { ... }

What am i missing? Thanks!

like image 863
metacortechs Avatar asked Mar 13 '12 19:03

metacortechs


2 Answers

Try to explicitly define the name for your CheckSession bean and see if that helps... Like this:

@Component("CheckSession")
public class CheckSession extends OncePerRequestFilter implements Filter {
    @Autowired private Usuario usuario;

    @Override
    protected void doFilterInternal(HttpServletRequest request,
            HttpServletResponse response, FilterChain chain)
            throws ServletException, IOException {

        //  always null
        System.out.println("autowired " + usuario);
        chain.doFilter(request,  response);
    }
}

The key part is this: @Component("CheckSession")

And to make things prettier and easier to deal with down the road, I would camelCase the name and rename it to "checkSession" everywhere (de-capitalize first letter).

like image 124
anton1980 Avatar answered Oct 17 '22 14:10

anton1980


Works on Spring 3.2.6:

First: Name your component, 2 options - do only one

Option A: Annotation - assumes component scanning enabled

@Component("checkSession")
public class CheckSession extends OncePerRequestFilter {

@Autowired 
Usuario usuario;

@Override
protected void doFilterInternal(final HttpServletRequest request, final HttpServletResponse response, final FilterChain chain) throws ServletException, IOException {

    // not null
    System.out.println("autowired " + usuario);
    chain.doFilter(request,  response);
    }
}

Option B: wire bean via appliationContext.xml

<bean id="checkSession" class="com.example.filter.CheckSession"></bean>

Second: Wire up Spring DelegatingFilterProxy

<filter>
  <filter-name>checkSession</filter-name>
  <filter-class>org.springframework.web.filter.DelegatingFilterProxy</filter-class>
</filter>

<filter-mapping>
  <filter-name>checkSession</filter-name>
  <url-pattern>/*</url-pattern>
</filter-mapping>

Notes: the bean name is referenced as the filter-name

like image 28
jurassix Avatar answered Oct 17 '22 14:10

jurassix