New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat: add keep-alive settings #22

Merged

Tienisto merged 4 commits into Tienisto:main from nicobritos:keep-alive

Sep 17, 2024

Contributor

nicobritos commented Sep 17, 2024 •

edited

Loading

Add keep alive timeout settings to enable KA on reqwest, both in HTTP1 (normal TCP KA) and in HTTP2 (allowing KA on idle).

Also, deprecates usage of timeout and connectTimeout to move both of those settings to TimeoutSettings, which includes KA settings.

EDIT: This is needed because by default reqwest doesn't use KA by default:

KA on HTTP2 when idle (default is false)
HTTP2 KA ping (default is None)
HTTP2 KA timeout (default is None, disable KA)
KA on HTTP1 (I understand that it's disabled by default)

nicobritos and others added 4 commits

September 17, 2024 10:36


          feat: add keep-alive settings

2f6b62f


          deps: fix pubspec.lock

62163cd


          docs: typo

2c7d84a


          docs: add slash between HTTP and version

61df1fc

Tienisto merged commit a7a5521 into Tienisto:main

2 checks passed

Owner

Tienisto commented Sep 17, 2024

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet