CJCombrink commented on code in PR #3164:
URL: https://github.com/apache/thrift/pull/3164#discussion_r2131626677
##########
test/cpp/src/TestClient.cpp:
##########
@@ -321,9 +322,15 @@ int main(int argc, char** argv) {
std::shared_ptr<TProtocol> protocol2; // SecondService for multiplexed
if (ssl) {
- cout << "Client Certificate File: " << certPath << '\n';
- cout << "Client Key File: " << keyPath << '\n';
- cout << "CA File: " << caPath << '\n';
+ auto fileExists = [](const std::string& path) {
+ std::ifstream f(path.c_str());
+ return f.good();
+ };
+
+ cout << "Client Path : " << testDir << '\n';
+ cout << "Client Certificate File: " << certPath << " (" << std::boolalpha
<< fileExists(certPath) << ")"<< '\n';
Review Comment:
Trying to debug SSL and cert issues I though the certs are missing in the
pipeline.
Concluded through this code that it was not the issue and decided to keep
this here since it can be useful in future
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
