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
ssl: adjust cacertfile option #6287
ssl: adjust cacertfile option #6287
Conversation
u3s
commented
Sep 8, 2022
- perform conversions before loading cert data from file
- convert relative path to absolute for avoiding collisions
- convert symlink to actual file path
CT Test Results 2 files 64 suites 47m 45s ⏱️ Results for commit 443696e. To speed up review, make sure that you have read Contributing to Erlang/OTP and that all checks pass. See the TESTING and DEVELOPMENT HowTo guides for details about how to run test locally. Artifacts// Erlang/OTP Github Action Bot |
@@ -390,12 +390,12 @@ alternative_path_hardlink(Config) when is_list(Config) -> | |||
alternative_path_symlink() -> | |||
[{doc,"Test that internal reference table contains expected data for" | |||
" absolute and symbolic link. " | |||
"This test verifies handling of same file with an alternative reference."}]. | |||
%% see alternative_path_hardlink for specification | |||
"This test verifies handling of same file with an alternative reference." |
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.
Maybe add some explanation of why counters change in the way they do!? See for instance the manual clean-up case.
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.
I tried to improve the comment.
443696e
to
3573d89
Compare
- perform conversions before loading cert data from file - convert relative path to absolute for avoiding collisions - convert symlink to actual file path
3573d89
to
8c7fa30
Compare