fix akka#4213: Content-Length when entity is unallowed

bursauxa · Jan 12, 2023 · 471e6ca · 471e6ca
1 parent 84af061
commit 471e6ca
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/...ttp-core/src/main/scala/akka/http/impl/engine/rendering/HttpResponseRendererFactory.scala b/...ttp-core/src/main/scala/akka/http/impl/engine/rendering/HttpResponseRendererFactory.scala
@@ -234,7 +234,7 @@ private[http] class HttpResponseRendererFactory(
           }
 
           def renderContentLengthHeader(contentLength: Long) =
-            if (status.allowsEntity) r ~~ ContentLengthBytes ~~ contentLength ~~ CrLf else r
+            if (status.allowsEntity) r ~~ ContentLengthBytes ~~ contentLength ~~ CrLf else r ~~ ContentLengthBytes ~~ 0 ~~ CrLf
 
           def headersAndEntity(entityBytes: => Source[ByteString, Any]): StrictOrStreamed =
             if (noEntity) {

diff --git a/akka-http-core/src/test/scala/akka/http/impl/engine/rendering/ResponseRendererSpec.scala b/akka-http-core/src/test/scala/akka/http/impl/engine/rendering/ResponseRendererSpec.scala
@@ -65,6 +65,7 @@ class ResponseRendererSpec extends AnyFreeSpec with Matchers with BeforeAndAfter
             |Age: 0
             |Server: akka-http/1.0.0
             |Date: Thu, 25 Aug 2011 09:10:29 GMT
+            |Content-Length: 0
             |
             |"""
         }