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.

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

fix: fix mtls issue in handwritten layer #226

merged 3 commits into from Oct 22, 2020

fix: fix mtls issue in handwritten layer #226

merged 3 commits into from Oct 22, 2020


Copy link

@arithmetic1728 arithmetic1728 commented Oct 21, 2020

fix #224

GAPIC client has mtls logic, which determines the api endpoint (mtls endpoint vs regular endpoint) and ssl credentials for the channel. The handwritten layer client creates its own channel, and mtls logic is completely missing. The solution is: creating a helper GAPIC client, which uses its own mtls logic to determine the mtls info (api endpoint and ssl credentials), then we can use these mtls info in the handwritten layer client initialization.
The changes are in the following 4 files:

All the other files are re-generated with synthtool (they are some test/sample scripts etc).

@arithmetic1728 arithmetic1728 changed the title Mtls fix: fix mtls issue in handwritten layer Oct 21, 2020
@arithmetic1728 arithmetic1728 marked this pull request as ready for review Oct 21, 2020
@arithmetic1728 arithmetic1728 requested review from as code owners Oct 21, 2020
Copy link

@anguillanneuf anguillanneuf left a comment

Would you mind reverting the lint style changes in

  • scripts/
  • samples/snippets/
    As those aren't needed.


CONTRIBUTING.rst Outdated Show resolved Hide resolved
@anguillanneuf anguillanneuf self-requested a review Oct 21, 2020
@arithmetic1728 arithmetic1728 requested a review from busunkim96 Oct 21, 2020
@arithmetic1728 arithmetic1728 merged commit 09a409c into master Oct 22, 2020
8 checks passed
@arithmetic1728 arithmetic1728 deleted the mtls branch Oct 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet
Linked issues

Successfully merging this pull request may close these issues.

4 participants