flush outputStream if status >= 400

SentryMan · SentryMan · commit c8dddac2a10c · 2025-03-22T14:30:50.000-04:00
diff --git a/src/main/java/robaho/net/httpserver/ExchangeImpl.java b/src/main/java/robaho/net/httpserver/ExchangeImpl.java
@@ -221,7 +221,6 @@ PlaceholderOutputStream getPlaceholderResponseBody() {
     
     public void sendResponseHeaders(int rCode, long contentLen)
             throws IOException {
-        uis.close();
         final Logger logger = getServerImpl().getLogger();
         if (sentHeaders) {
             throw new IOException("headers already sent");
diff --git a/src/main/java/robaho/net/httpserver/FixedLengthOutputStream.java b/src/main/java/robaho/net/httpserver/FixedLengthOutputStream.java
@@ -90,7 +90,7 @@ public void close() throws IOException {
         // if after reading the rest of the known input for this request, there is
         // more input available, http pipelining is in effect, so avoid flush, since
         // it will be flushed after processing the next request
-        if(is.getRawInputStream().available()==0) {
+        if(is.getRawInputStream().available() == 0 || t.rcode >= 400) {
             flush();
         }
 

Original file line number	Diff line number	Diff line change
`@@ -90,7 +90,7 @@ public void close() throws IOException {`
`90`	`90`	`// if after reading the rest of the known input for this request, there is`
`91`	`91`	`// more input available, http pipelining is in effect, so avoid flush, since`
`92`	`92`	`// it will be flushed after processing the next request`
`93`		`- if(is.getRawInputStream().available()==0) {`
	`93`	`+ if(is.getRawInputStream().available() == 0 \|\| t.rcode >= 400) {`
`94`	`94`	`flush();`
`95`	`95`	`}`
`96`	`96`