-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
add sendTimeout and receiveTimeout in Http2Adapter #1942
Conversation
} else { | ||
await requestStreamFuture; | ||
} |
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.
TBH I'd prefer how the IO adapter wrote, future = future.timeout
, etc.
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.
I prefer final variable rather than reassigning it
Co-authored-by: Alex Li <github@alexv525.com> Signed-off-by: sunhapper <9249052+sunhapper@users.noreply.github.com>
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. Thanks for the contribution again.
Add send time out and receive time for http2
New Pull Request Checklist
main
branch to avoid conflicts (via merge from master or rebase)CHANGELOG.md
in the corresponding packageAdditional context and info (if any)
Http2Adapter only implements connect timeout,the
RequestOptions.sendTimeout
andRequestOptions.receiveTimeout
not work.