-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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 tck ssl client #3471
add tck ssl client #3471
Conversation
Codecov Report
@@ Coverage Diff @@
## master #3471 +/- ##
==========================================
- Coverage 85.40% 85.24% -0.17%
==========================================
Files 1289 1304 +15
Lines 119798 121239 +1441
==========================================
+ Hits 102317 103347 +1030
- Misses 17481 17892 +411
Continue to review full report at Codecov.
|
62391c0
to
3860c3a
Compare
fbf2363
to
4e66d52
Compare
@@ -450,3 +455,18 @@ def parse_service_index(name: str): | |||
if m and len(m.groups()) == 2: | |||
return int(m.groups()[1]) | |||
return None | |||
|
|||
def get_ssl_config(is_graph_ssl: bool, ca_signed: bool): |
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.
Why add this function?
_params['ca_path'] = 'share/resources/test.ca.pem' | ||
_params['cert_path'] = 'share/resources/test.ca.key' | ||
_params['key_path'] = 'share/resources/test.ca.password' | ||
_params['cert_path'] = 'share/resources/test.derive.crt' |
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.
_params['cert_path'] = 'share/resources/test.ca.pem'
_params['key_path'] = 'share/resources/test.ca.key'
_params['password_path'] = 'share/resources/test.ca.password'
What type of PR is this?
What does this PR do?
use ssl client in tck test
Close #3433
should not be merged utils nebula-python support
password