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
mobile: Remove support for YAML in the mobile APIs #33799
Conversation
Signed-off-by: Ali Beyad <abeyad@google.com>
/wait on CI I think? |
yes, sorry, i'm still working on figuring out the QuicTestServerTest and put it on the backburner Friday for other stuff. I didn't realize this was assigned to you, let me reassign to myself until its ready |
Signed-off-by: Ali Beyad <abeyad@google.com>
Signed-off-by: Ali Beyad <abeyad@google.com>
Signed-off-by: Ali Beyad <abeyad@google.com>
/assign @fredyw |
It looks like the CI is still failing. |
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.
Nice! Thanks for working on this.
/retest |
/retest |
As part of this change, a new EngineBuilder
setUpstreamTlsSni()
API was added for all the language bindings,so that the upstream TLS socket's SNI can be overridden for test servers where the cert is created with a particular
hostname but we don't access the server via that hostname (instead, we use local address and local port).