PR #12643 - move validation to core response

Signed-off-by: Lachlan Roberts <[email protected]>
jetty · Dec 16, 2024 · 92d5b92 · 92d5b92
1 parent 57eb041
commit 92d5b92
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/...y-core/jetty-server/src/main/java/org/eclipse/jetty/server/internal/HttpChannelState.java b/...y-core/jetty-server/src/main/java/org/eclipse/jetty/server/internal/HttpChannelState.java
@@ -1190,6 +1190,8 @@ public int getStatus()
         @Override
         public void setStatus(int code)
         {
+            if (code < 100 || code > 999)
+                throw new IllegalArgumentException();
             if (!isCommitted())
                 _status = code;
         }

diff --git a/...0/jetty-ee10-servlet/src/main/java/org/eclipse/jetty/ee10/servlet/ServletApiResponse.java b/...0/jetty-ee10-servlet/src/main/java/org/eclipse/jetty/ee10/servlet/ServletApiResponse.java
@@ -250,8 +250,6 @@ public void addIntHeader(String name, int value)
     @Override
     public void setStatus(int sc)
     {
-        if (sc < 100 || sc > 999)
-            throw new IllegalArgumentException();
         getResponse().setStatus(sc);
     }