-
Notifications
You must be signed in to change notification settings - Fork 206
Fetch keepalive support. #963
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.
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
Conversation
4f94e02 to
cf23bec
Compare
7a19acc to
239eda9
Compare
VadimZhestikov
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good for me
63cf023 to
ee49384
Compare
VadimZhestikov
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good addition
hongzhidao
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
2535f1a
ee49384 to
2535f1a
Compare
e5cc78c
2535f1a to
e5cc78c
Compare
|
TODO: remove ip address match when looking for a connection to reuse. Use (scheme, host, port) tuple instead. |
054b721 to
4d46154
Compare
4d46154 to
4a4c3e7
Compare
VadimZhestikov
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good
This closes nginx#957 feature request on Github.
4a4c3e7 to
c33284d
Compare
No description provided.