Skip to content

Commit dd19ed3

Browse files
matthewstevenson88copybara-github
authored andcommitted
[ssl] Downgrade SSL handshake failure log to INFO. (grpc#38058)
This log can be hit under normal circumstances (e.g. a client has an expired cert and authenticates to the server), so this should be an INFO-level log rather than an ERROR-level log. Closes grpc#38058 COPYBARA_INTEGRATE_REVIEW=grpc#38058 from matthewstevenson88:downgrade23 1cbdd5a PiperOrigin-RevId: 693375018
1 parent 2a9c752 commit dd19ed3

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

src/core/tsi/ssl_transport_security.cc

+3-3
Original file line numberDiff line numberDiff line change
@@ -1662,9 +1662,9 @@ static tsi_result ssl_handshaker_do_handshake(tsi_ssl_handshaker* impl,
16621662
const char* verify_err = X509_verify_cert_error_string(verify_result);
16631663
verify_result_str = absl::StrCat(": ", verify_err);
16641664
}
1665-
LOG(ERROR) << "Handshake failed with fatal error "
1666-
<< grpc_core::SslErrorString(ssl_result) << ": " << err_str
1667-
<< verify_result_str;
1665+
LOG(INFO) << "Handshake failed with error "
1666+
<< grpc_core::SslErrorString(ssl_result) << ": " << err_str
1667+
<< verify_result_str;
16681668
if (error != nullptr) {
16691669
*error = absl::StrCat(grpc_core::SslErrorString(ssl_result), ": ",
16701670
err_str, verify_result_str);

0 commit comments

Comments
 (0)