maskit commented on pull request #7479:
URL: https://github.com/apache/trafficserver/pull/7479#issuecomment-777874810


   Should we have a setting value for OpenSSL impl? If we added it later, the 
value for it would be `2`, which would be confusing because `2` means ATS impl 
for client session cache.
   
   Is there any reason for disabling this by default? The one for client 
session cache is enabled by default and it uses ATS impl.
   
   We might need to call `SSL_CTX_sess_set_remove_cb()` as well. I'm not sure. 
OpenSSL API doc says it's for internal cache, but we set a callback for client 
session cache when we use ATS impl.
   
   Please add docs for the new settings.


----------------------------------------------------------------
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]


Reply via email to