Issue #4711 trailers

Standard trailers cannot be set if committed or HTTP/1.0 Signed-off-by: Greg Wilkins <gregw@webtide.com>
jetty · Apr 1, 2020 · af89596 · af89596
1 parent ec38e99
commit af89596
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/jetty-server/src/main/java/org/eclipse/jetty/server/Response.java b/jetty-server/src/main/java/org/eclipse/jetty/server/Response.java
@@ -1229,7 +1229,10 @@ public Supplier<Map<String, String>> getTrailerFields()
     @Override
     public void setTrailerFields(Supplier<Map<String, String>> trailers)
     {
-        // TODO new for 4.0 - avoid transient supplier?
+        if (isCommitted())
+            throw new IllegalStateException("Committed");
+        if (getHttpChannel().getRequest().getHttpVersion().ordinal() <= HttpVersion.HTTP_1_0.ordinal())
+            throw new IllegalStateException("Trailers not supported");
         this._trailers = new HttpFieldsSupplier(trailers);
     }