Skip to content

Commit

Permalink
Document the correct default value for HTTP/2 max concurrent streams (#…
Browse files Browse the repository at this point in the history
…82)

Load testing on our web application caused issues and it looks like it
is because of this.

The default value is 200 as you can see from the default instantiation
here: https://github.com/hyperium/hyper/blob/1d4ff3597b8e76818c8553dbfa4234cf4208c958/src/proto/h2/server.rs#L66
  • Loading branch information
psibi committed Dec 21, 2023
1 parent a5e34eb commit 4d1ed57
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/server/conn/auto.rs
Original file line number Diff line number Diff line change
Expand Up @@ -597,7 +597,7 @@ impl<E> Http2Builder<'_, E> {
/// Sets the [`SETTINGS_MAX_CONCURRENT_STREAMS`][spec] option for HTTP2
/// connections.
///
/// Default is no limit (`std::u32::MAX`). Passing `None` will do nothing.
/// Default is 200. Passing `None` will remove any limit.
///
/// [spec]: https://http2.github.io/http2-spec/#SETTINGS_MAX_CONCURRENT_STREAMS
pub fn max_concurrent_streams(&mut self, max: impl Into<Option<u32>>) -> &mut Self {
Expand Down

0 comments on commit 4d1ed57

Please sign in to comment.