Always set content-length

Not setting it makes some clients unhappy.
ponylang · Feb 4, 2024 · 2b8df83 · 2b8df83
1 parent 0802950
commit 2b8df83
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/.release-notes/set-content-length.md b/.release-notes/set-content-length.md
@@ -0,0 +1,3 @@
+## Always set Content-Length
+
+There was a bug where the Content-Length header could end up not being sent. This would cause many clients (such as curl) to hang.
diff --git a/http_server/response.pony b/http_server/response.pony
@@ -298,8 +298,8 @@ class val BuildableResponse is (Response & ByteSeqIter)
     match cl
     | let clu: USize =>
       set_header("Content-Length", cl.string())
-    // | None =>
-    // TODO: drop header
+    | None =>
+      set_header("Content-Length", 0)
     end
     this