duke8253 commented on pull request #7479: URL: https://github.com/apache/trafficserver/pull/7479#issuecomment-780024422
> Thanks for the testing. Seems like we don't need to register those callbacks. > > > > I found below on BoringSSL API doc. Seems like we don't need an option to use internal cache for client. > > > SSL_SESS_CACHE_CLIENT enables session caching for a client. The internal cache is never used on a client, so this only enables the callbacks. > > > > I don't have strong opinion on this but what's the reason to disable origin session reuse by default? It could be a surprise if a user upgrade ATS, but I guess it's acceptable if this is going to be released on ATS 10 (not 9.x). Originally I had it enabled by default, changed it to be disabled for testing purposes. I can go either way on this , I'll ask Susan during my weekly meeting today for her input. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
