AwsProxyServletContext Does Not Support Servlet Filter Registration about aws-serverless-java-container HOT 9 CLOSED

I did some more investigation and determined with more certainty that indeed, the current implementation does not support Filter registration (or Servlet registration, for that matter). I also determined that HandlerInterceptors will not serve my particular need, so I am just going to work around this in the handleRequest method of my own project for the time being, essentially bypassing Spring for part of my code.

I probably will not have time to patch this library to add Filter support myself any time soon, so unfortunately I can't offer a Pull Request at the moment. I'll leave the ticket open for now in case anyone else is interested in this as a feature, but I'm also okay with it if this is a "Won't Fix." Thanks!

from aws-serverless-java-container.

Correct, at the moment it does not. I'd be interested to take a look at your filter code and see if/how we can add support for it. Anything you can share on GitHub or here?

from aws-serverless-java-container.

Is this also the reason why '@Valid' isn't working?

from aws-serverless-java-container.

I'm no spring expert but doing some research I can see that the @Valid annotation does run as a filter. We are currently designing filters support based on this issue and we'll have something in a branch over the next few weeks.

from aws-serverless-java-container.

Thanks for the answer @sapessi ..

I created a separated issue for this...

from aws-serverless-java-container.

I'll keep that issue open until I can confirm for certain that this is the cause of the issue. If it is, then I'll close the separate issue and group everything in here. Otherwise we'll address it there.

from aws-serverless-java-container.

Hey @blakeney, could you share some sample code on how you initialize (and add) filters in your spring application?

from aws-serverless-java-container.

I've pushed a first version of the filter support to the servlet-improvements branch. This is the commit d7bec0a. You can register filters with the setStartupHandler lambda:

try {
    handler = SpringLambdaContainerHandler.getAwsProxyHandler(EchoSpringAppConfig.class);
    handler.setStartupHandler(c -> {
        FilterRegistration.Dynamic registration = c.addFilter("CustomHeaderFilter", CustomHeaderFilter.class);
        // update the registration to map to a path
        registration.addMappingForUrlPatterns(EnumSet.of(DispatcherType.REQUEST), true, "/*");
        // servlet name mappings are disabled and will throw an exception
    });
} catch (ContainerInitializationException e) {
    e.printStackTrace();
}

Next I will work on supporting the filter annotations.

from aws-serverless-java-container.

Version 0.4 is out and adds support for servlet filter:

handler.setStartupHandler(c -> {
    FilterRegistration.Dynamic registration = c.addFilter("CustomHeaderFilter", CustomHeaderFilter.class);
    // update the registration to map to a path
    registration.addMappingForUrlPatterns(EnumSet.of(DispatcherType.REQUEST), true, "/*");
    // servlet name mappings are disabled and will throw an exception
});

Closing this issue.

from aws-serverless-java-container.