Merge pull request #597 from jooby-project/596

unhandled requests must returns a 404 not 415 fix #596

Author: jknack

coverage-report/src/test/java/org/jooby/issues/Issue596.java

@@ -0,0 +1,20 @@
+package org.jooby.issues;
+
+import org.jooby.test.ServerFeature;
+import org.junit.Test;
+
+public class Issue596 extends ServerFeature {
+
+  {
+    get("/596", (req, rsp) -> {
+
+    });
+  }
+
+  @Test
+  public void shouldGet404onUnhandledRequests() throws Exception {
+    request()
+        .get("/596")
+        .expect(404);
+  }
+}

jooby/src/main/java/org/jooby/Status.java

@@ -22,9 +22,8 @@
  * HTTP status codes.
  *
  * <p>
- * This code has been kindly borrowed from <a href="http://spring.io/">Spring</a>
+ * This code has been kindly borrowed from <a href="http://spring.io/">Spring</a>.
  * </p>
- * .
  *
  * @author Arjen Poutsma
  * @see <a href="http://www.iana.org/assignments/http-status-codes">HTTP Status Code Registry</a>

jooby/src/main/java/org/jooby/internal/HttpHandlerImpl.java

@@ -503,7 +503,9 @@ private static Err handle406or415(final Set<Route.Definition> routeDefs, final S
               .map(MediaType::name)
               .collect(Collectors.joining(", ")));
         }
-        return new Err(Status.UNSUPPORTED_MEDIA_TYPE, contentType.name());
+        if (!contentType.isAny()) {
+          return new Err(Status.UNSUPPORTED_MEDIA_TYPE, contentType.name());
+        }
       }
     }
     return null;