mirrors/actix-web
1
0
Fork 0
mirror of https://github.com/actix/actix-web.git synced 2025-01-20 22:18:06 +00:00
Code Issues Projects Releases Wiki Activity

fix(http2): remove host header when using http2

Browse source
This commit is contained in:
Joel Wurtz 2024-12-07 16:08:35 +01:00 committed by Joel Wurtz
parent 8115c818c1
commit 12107cdc5c
No known key found for this signature in database
GPG key ID: ED264D1967A51B0D
2 changed files with 3 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
awc/CHANGES.md
View file

@ -5,6 +5,7 @@
- Update `brotli` dependency to `7`. - Update `brotli` dependency to `7`.
- Prevent panics on connection pool drop when Tokio runtime is shutdown early. - Prevent panics on connection pool drop when Tokio runtime is shutdown early.
- Minimum supported Rust version (MSRV) is now 1.75. - Minimum supported Rust version (MSRV) is now 1.75.
- Do not send `Host` header on HTTP/2 requests, as it is not required, and some web servers may reject it.
## 3.5.1 ## 3.5.1

4
awc/src/client/h2proto.rs
View file

@ -12,7 +12,7 @@ use h2::{
SendStream, SendStream,
}; };
use http::{ use http::{
header::{HeaderValue, CONNECTION, CONTENT_LENGTH, TRANSFER_ENCODING}, header::{HeaderValue, CONNECTION, CONTENT_LENGTH, HOST, TRANSFER_ENCODING},
request::Request, request::Request,
Method, Version, Method, Version,
}; };
@ -97,7 +97,7 @@ where
// TODO: consider skipping other headers according to: // TODO: consider skipping other headers according to:
// https://datatracker.ietf.org/doc/html/rfc7540#section-8.1.2.2 // https://datatracker.ietf.org/doc/html/rfc7540#section-8.1.2.2
// omit HTTP/1.x only headers // omit HTTP/1.x only headers
CONNECTION | TRANSFER_ENCODING => continue, CONNECTION | TRANSFER_ENCODING | HOST => continue,
CONTENT_LENGTH if skip_len => continue, CONTENT_LENGTH if skip_len => continue,
// DATE => has_date = true, // DATE => has_date = true,
_ => {} _ => {}