Skip to content
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

Client should respect the "Keep-Alive" header #2720

Open
loyd opened this issue Dec 10, 2021 · 2 comments
Open

Client should respect the "Keep-Alive" header #2720

loyd opened this issue Dec 10, 2021 · 2 comments
Labels
C-feature Category: feature. This is adding a new feature.

Comments

@loyd
Copy link

loyd commented Dec 10, 2021

It seems that hyper doesn't respect the Keep-Alive header and just uses a predefined value (90 seconds). Instead, hyper should consider the header and rotate connections according to it.

Related issue: #2136. Users disable the pool totally as the easiest solution.
@loyd loyd added the C-feature Category: feature. This is adding a new feature. label Dec 10, 2021
@seanmonstar
Copy link
Member

Do other client libraries pay attention to that header? I haven't noticed any...

@sfackler
Copy link
Contributor

sfackler commented Jan 1, 2022

It looks like browsers universally support it at least: https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Keep-Alive

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
C-feature Category: feature. This is adding a new feature.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@seanmonstar @loyd @sfackler