stoty commented on PR #2270: URL: https://github.com/apache/zookeeper/pull/2270#issuecomment-3082667525
Thanks for the review @anmolnar . It turns out that I have misread the tcnative/boringssl code, and current BoringSSL DOES support ocsp stapling. I have removed the comments and docs stating that BoringSSL does not support stapling. In any case, I have added checks for OpenSSL.isOcspSupported() and refactored the duplicate logic into a new method. I have also added more logging and tried to make the logic simpler and easier to follow. -- 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: notifications-unsubscr...@zookeeper.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
