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

Add connect_timeout to EndpointConfig #89

Closed
akiradeveloper opened this issue Nov 24, 2020 · 3 comments
Closed

Add connect_timeout to EndpointConfig #89

akiradeveloper opened this issue Nov 24, 2020 · 3 comments
Labels
enhancement New feature or request

Comments

@akiradeveloper
Copy link
Owner

akiradeveloper commented Nov 24, 2020 •
edited
Loading

Now EndpointConfig is allowed to set timeout but it is about entire request, not about connection. In some request what is needed is connection timeout rather than request timeout.

The current tonic doesn't support this feature but it is technically possible and would be available when hyperium/tonic#498 is accomplished.
@akiradeveloper akiradeveloper added the enhancement New feature or request label Nov 24, 2020
@akiradeveloper
Copy link
Owner Author

I thought hyperium/tonic#481 will also solve this issue but the wrapper around Channel only affects things after-connection. Of course, my concern here is before-connection.

@akiradeveloper
Copy link
Owner Author

This feature is marked as 0.4 in tonic. Will wait.

@akiradeveloper
Copy link
Owner Author

There is nothing I can do for this. Now close.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@akiradeveloper