-
Notifications
You must be signed in to change notification settings - Fork 10.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 GRPC_ARG_HTTP_PROXY channel argument. #20866
Conversation
Introduce the GRPC_ARG_HTTP_PROXY channel argument so that HTTP proxy settings can be controlled per channel.
|
- Clarify comments - Simplify code by using grpc_channel_args_find_string function
Sorry for the delay; this somehow slipped off my radar. The changes look good, but it looks like there's a merge conflict to address. |
I've resolved the merge conflicts, noticed that one of the Sanity Checks was failing, and fixed the formatting error. Hopefully should be ready to merge now. |
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.
This change looks good to me.
I've triggered the tests to run. Yash, if everything looks good with the tests, please feel free to merge.
This issue/PR has been automatically marked as stale because it has not had any update (including commits, comments, labels, milestones, etc) for 30 days. It will be closed automatically if no further update occurs in 7 day. Thank you for your contributions! |
Yash, can you please check the tests and get this merged? Thanks! |
Introduce the GRPC_ARG_HTTP_PROXY channel argument so that HTTP proxy
settings can be controlled per channel.
@yashykt