Fix `maximum_connection_streams` -> `@remote_settings`. - Githubissues

socketry / protocol-http2

MIT License

9 stars 10 forks source link

Fix `maximum_connection_streams` -> `@remote_settings`. #18

Closed ioquatix closed 3 months ago

ioquatix commented 3 months ago

Connection#maximum_concurrent_streams was using @local_settings, but it is used for the concurrency limit in async-http. This is simply the wrong value to use, it should be using the @remote_settings.

Fixes https://github.com/socketry/async-http/issues/161.

Types of Changes

Bug fix.

Contribution

[x] I added tests for my changes.
[x] I tested my changes locally.
[x] I agree to the Developer's Certificate of Origin 1.1.