fix: drop rendered headers containing NUL as well as CR and LF

[pjfanning]

Motivation

HTTP 1.1 equivalent of #1258

The guard in Rendering.~~(HttpHeader) renders a header and then scans the rendered bytes, discarding it if it finds a character that must never reach the wire. It only looked for CR and LF:

if (ch == '\r' || ch == '\n') { /* discard */ }

so a RawHeader value carrying a NUL was rendered as-is. NUL is not a legal field-value character, and a downstream consumer that treats the value as a C string truncates it there — so two parties can disagree about where the value ends. The HTTP/2 renderer rejects CR, LF and NUL alike (see #1258), which left HTTP/1.1 as the weaker of the two.

Modification

Add Rendering.isIllegalHeaderChar, covering CR, LF and NUL, and call it from all four check implementations — StringRendering, ByteArrayRendering, ByteStringRendering and CustomCharsetByteStringRendering — rather than repeating the character comparison a fourth time. It takes an Int so the Char- and Byte-based renderings can pass their element straight in.

Result

A header whose name or value contains NUL is discarded like one containing CR or LF, on every rendering implementation, and the rule now lives in one place.

Tests

• sbt "http-core/testOnly org.apache.pekko.http.impl.util.RenderingSpec org.apache.pekko.http.impl.engine.rendering.ResponseRendererSpec org.apache.pekko.http.impl.engine.rendering.RequestRendererSpec" — pass (80 tests). A new case in the shared rendering table asserts a header with NUL in the value is discarded. Verified it fails with the fix stashed — once for each of the four renderings.
• sbt http-core/mimaReportBinaryIssues — pass (internal impl.util change, no public API).
• Native scalafmt on the changed files — clean.

References

None - extends the outgoing header guard to NUL

🤖 Generated with Claude Code