tree f6a0d71699455bc09351f92082ce3d8309702e65
parent 471a0632ce727f6824f06939532b767f8782ad53
author jprat <jprat@google.com> 1787066149 -0700
committer Copybara-Service <copybara-worker@google.com> 1787066191 -0700

Fix `BalsaFrame` token corruption when sanitizing first line with multiple spaces.

**Key Changes:**

- Fixes a bug in `BalsaFrame::ParseHTTPFirstLine` where token extraction arithmetic applied indices from a new, sanitized storage block to the pointer of the original, un-sanitized raw buffer block. This yielded corrupted parser tokens (e.g., getting `" "` instead of `"/"` for a URL in `GET  /  HTTP/1.1`).
- Passes `begin` by reference `(char*&)` to update it to the new buffer allocation, ensuring indices remain perfectly tied to their active memory block container.
- Updates `balsa_frame_test.cc` to explicitly verify visitor callbacks (`OnRequestFirstLineInput` / `OnResponseFirstLineInput`) receive clean, uncorrupted component data.
- Adds `SanitizeFirstlineSpaces_Request_Served` and `SanitizeFirstlineSpaces_Response_Served` to Envoy integration tests to verify the hook works seamlessly at the gateway level.

Protected by sanitize_firstline_spaces (The flag is not in production yet).

PiperOrigin-RevId: 966603762