Skip to content

test: add kitex grpc client with gPRC server TLS test - #47

Open
ppzqh wants to merge 7 commits into
mainfrom
test/add_grpc_tls
Open

test: add kitex grpc client with gPRC server TLS test#47
ppzqh wants to merge 7 commits into
mainfrom
test/add_grpc_tls

Conversation

@ppzqh

@ppzqh ppzqh commented Jan 11, 2024

Copy link
Copy Markdown
Contributor

What type of PR is this?

test

Check the PR title.

  • This PR title match the format: <type>(optional scope): <description>
  • The description of this PR title is user-oriented and clear enough for others to understand.
  • Attach the PR updating the user documentation if the current PR requires user awareness at the usage level. User docs repo

(Optional) Translate the PR title into Chinese.

zh: test: 添加 kitex grpc client 与 gRPC server 的 TLS 互通测试

(Optional) More detailed description for this PR(en: English/zh: Chinese).

en:
zh(optional):

(Optional) Which issue(s) this PR fixes:

(optional) The PR that updates user documentation:

@ppzqh
ppzqh requested review from a team as code owners January 11, 2024 07:51

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

license-eye has totally checked 124 files.

Valid Invalid Ignored Fixed
80 2 42 0
Click to see the invalid file list
  • grpc/tls/tls_test.go
  • grpc/tls/cert/gen.sh

Comment thread grpc/tls/cert/gen.sh
Comment thread grpc/tls/tls_test.go
@ppzqh
ppzqh force-pushed the test/add_grpc_tls branch 3 times, most recently from 85cd5e2 to e2482d3 Compare January 11, 2024 08:03
@CLAassistant

CLAassistant commented Dec 10, 2024

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
2 out of 3 committers have signed the CLA.

✅ YangruiEmma
✅ ppzqh
❌ Felix021


Felix021 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants