duke8253 commented on code in PR #10115:
URL: https://github.com/apache/trafficserver/pull/10115#discussion_r1279502336
##########
iocore/net/SSLNetVConnection.cc:
##########
@@ -2511,7 +2545,9 @@ SSLNetVConnection::update_early_data_config(uint32_t
max_early_data, uint32_t re
#else
// If SSL_set_max_early_data is unavailable, it's probably BoringSSL,
// and SSL_set_early_data_enabled should be available.
- SSL_set_early_data_enabled(ssl, max_early_data > 0 ? 1 : 0);
+ bool const early_data_enabled = max_early_data > 0 ? 1 : 0;
+ SSL_set_early_data_enabled(ssl, early_data_enabled);
+ Debug("ssl", "Called SSL_set_early_data_enabled with %d",
early_data_enabled);
Warning("max_early_data is not used due to library limitations");
Review Comment:
Yes, I think this can be removed here.
--
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]
