Hummingbird 2's semantics changed, requiring body streams to finish the stream explicitly. This currently causes crashes on reused connections

Joannis · Joannis · commit e268d15e9da4 · 2024-09-30T11:50:42.000+02:00
diff --git a/Sources/OpenAPIHummingbird/OpenAPITransport.swift b/Sources/OpenAPIHummingbird/OpenAPITransport.swift
@@ -85,6 +85,7 @@ extension Response {
                     for try await buffer in bufferSequence {
                         try await writer.write(buffer)
                     }
+                    try await writer.finish(nil)
                 }
             } else {
                 responseBody = .init(asyncSequence: bufferSequence)

Original file line number	Diff line number	Diff line change
`@@ -85,6 +85,7 @@ extension Response {`
`85`	`85`	`for try await buffer in bufferSequence {`
`86`	`86`	`try await writer.write(buffer)`
`87`	`87`	`}`
	`88`	`+ try await writer.finish(nil)`
`88`	`89`	`}`
`89`	`90`	`} else {`
`90`	`91`	`responseBody = .init(asyncSequence: bufferSequence)`