Merge pull request #739 from jooby-project/730

Jooby overriding Content-Type header fix #730

Author: jknack

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

@@ -0,0 +1,23 @@
+package org.jooby.issues;
+
+import org.jooby.test.ServerFeature;
+import org.junit.Test;
+
+public class Issue730 extends ServerFeature {
+
+  {
+    get("/730", (req, rsp) -> {
+      rsp.header("content-type", "application/json");
+      rsp.send("{\"issue\": \"730\"}");
+    });
+
+  }
+
+  @Test
+  public void contentTypeShouldWorkFromSetHeader() throws Exception {
+    request().get("/730")
+        .execute()
+        .header("Content-Type", "application/json;charset=utf-8");
+  }
+
+}

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

@@ -446,6 +446,10 @@ private Response setHeader(final String name, final Object value) {
           }
           rsp.header(LOCATION, location);
         } else {
+          if ("Content-Type".equalsIgnoreCase(name)) {
+            // keep type reference
+            this.type = MediaType.valueOf(value.toString());
+          }
           rsp.header(name, Headers.encode(value));
         }
       }