Stop sending Host header in MASQUE HTTP/2 requests
In HTTP/2, the `:authority` header is mandatory and the `host` header is optional. The only requirement is that, if `host` is included, it needs to match what's in `:authority`. So this code was doing something allowed by [the spec](https://www.rfc-editor.org/rfc/rfc9113#section-8.3.1-2.3.1), but some servers fail to process it. Since this is just a test client, we'll work around that issue by only sending `:authority`.
PiperOrigin-RevId: 782086853
diff --git a/quiche/quic/masque/masque_ohttp_client_bin.cc b/quiche/quic/masque/masque_ohttp_client_bin.cc
index df2aca2..f7185de 100644
--- a/quiche/quic/masque/masque_ohttp_client_bin.cc
+++ b/quiche/quic/masque/masque_ohttp_client_bin.cc
@@ -165,7 +165,6 @@
request.headers[":scheme"] = url.scheme();
request.headers[":authority"] = url.HostPort();
request.headers[":path"] = url.path();
- request.headers["host"] = url.HostPort();
request.headers["accept"] = "application/ohttp-keys";
request.headers["content-type"] = "application/ohttp-keys";
absl::StatusOr<RequestId> request_id =
@@ -268,7 +267,6 @@
request.headers[":scheme"] = relay_url_.scheme();
request.headers[":authority"] = relay_url_.HostPort();
request.headers[":path"] = relay_url_.path();
- request.headers["host"] = relay_url_.HostPort();
request.headers["content-type"] = "message/ohttp-req";
request.body = ohttp_request->EncapsulateAndSerialize();
absl::StatusOr<RequestId> request_id =
diff --git a/quiche/quic/masque/masque_tcp_client_bin.cc b/quiche/quic/masque/masque_tcp_client_bin.cc
index 3f9d774..fc57693 100644
--- a/quiche/quic/masque/masque_tcp_client_bin.cc
+++ b/quiche/quic/masque/masque_tcp_client_bin.cc
@@ -476,7 +476,6 @@
headers[":scheme"] = url_.scheme();
headers[":authority"] = url_.HostPort();
headers[":path"] = url_.path();
- headers["host"] = url_.HostPort();
stream_id_ = h2_connection_->SendRequest(headers, std::string());
h2_connection_->AttemptToSend();
if (stream_id_ >= 0) {
