Added unit test to verify that issue #158 is actually supported in the framework

Author: sapessi

aws-serverless-java-container-spring/src/test/java/com/amazonaws/serverless/proxy/spring/SpringBootAppTest.java

@@ -30,6 +30,7 @@ public void testMethod_springSecurity_doesNotThrowException() {
         AwsProxyRequest req = new AwsProxyRequestBuilder("/test", "GET").build();
         AwsProxyResponse resp = handler.handleRequest(req, context);
         assertNotNull(resp);
+        assertEquals(200, resp.getStatusCode());
         validateSingleValueModel(resp, TestController.TEST_VALUE);
     }
 
@@ -53,7 +54,16 @@ public void defaultError_requestForward_springBootForwardsToDefaultErrorPage() {
             e.printStackTrace();
             fail();
         }
+    }
+
+    @Test
+    public void requestUri_dotInPathParam_expectRoutingToMethod() {
+        AwsProxyRequest req = new AwsProxyRequestBuilder("/test/testdomain.com", "GET").build();
 
+        AwsProxyResponse resp = handler.handleRequest(req, context);
+        assertNotNull(resp);
+        assertEquals(200, resp.getStatusCode());
+        validateSingleValueModel(resp, "testdomain.com");
     }
 
     private void validateSingleValueModel(AwsProxyResponse output, String value) {

aws-serverless-java-container-spring/src/test/java/com/amazonaws/serverless/proxy/spring/springbootapp/TestController.java

@@ -3,26 +3,56 @@
 
 import com.amazonaws.serverless.proxy.spring.echoapp.model.SingleValueModel;
 
+import org.springframework.context.annotation.Configuration;
+import org.springframework.http.MediaType;
 import org.springframework.security.config.annotation.web.builders.HttpSecurity;
 import org.springframework.security.config.annotation.web.configuration.EnableWebSecurity;
 import org.springframework.security.config.annotation.web.configuration.WebSecurityConfigurerAdapter;
+import org.springframework.web.accept.ContentNegotiationStrategy;
+import org.springframework.web.bind.annotation.PathVariable;
 import org.springframework.web.bind.annotation.RequestMapping;
 import org.springframework.web.bind.annotation.RequestMethod;
 import org.springframework.web.bind.annotation.RestController;
+import org.springframework.web.servlet.config.annotation.ContentNegotiationConfigurer;
+import org.springframework.web.servlet.config.annotation.EnableWebMvc;
+import org.springframework.web.servlet.config.annotation.PathMatchConfigurer;
+import org.springframework.web.servlet.config.annotation.WebMvcConfigurer;
+import org.springframework.web.servlet.config.annotation.WebMvcConfigurerAdapter;
 
 
 @RestController
 @EnableWebSecurity
-public class TestController extends WebSecurityConfigurerAdapter{
+public class TestController extends WebSecurityConfigurerAdapter {
     public static final String TEST_VALUE = "test";
 
+    // workaround to address the most annoying issue in the world: https://blog.georgovassilis.com/2015/10/29/spring-mvc-rest-controller-says-406-when-emails-are-part-url-path/
+    @Configuration
+    public static class CustomConfig extends WebMvcConfigurerAdapter {
+        @Override
+        public void configurePathMatch(PathMatchConfigurer configurer) {
+            configurer.setUseSuffixPatternMatch(false);
+        }
+
+        @Override
+        public void configureContentNegotiation(ContentNegotiationConfigurer configurer) {
+            configurer.favorPathExtension(false);
+        }
+    }
+
     @RequestMapping(path = "/test", method = { RequestMethod.GET })
     public SingleValueModel testGet() {
         SingleValueModel value = new SingleValueModel();
         value.setValue(TEST_VALUE);
         return value;
     }
 
+    @RequestMapping(path = "/test/{domain}", method = { RequestMethod.GET})
+    public SingleValueModel testDomainInPath(@PathVariable("domain") String domainName) {
+        SingleValueModel value = new SingleValueModel();
+        value.setValue(domainName);
+        return value;
+    }
+
     @Override
     protected void configure(HttpSecurity http) throws Exception {
         http.sessionManagement().disable();