Hello!
  I'd like to propose adding a new API get grab the SNI from the client 
connection.

const char * TSSslSNIGet(TSVConn sslp, int *length)

This would remove some of the redundant code in the rate_limit plugin but also 
would allow for the rate_limit plugin to be used under BoringSSL. The APIs 
between OpenSSL and BoringSSL here are pretty different here and don't have 
access to the same underlying structs. We already save off the name in the core 
(for both open and boring) and this API just exposes that value.

Here is the PR showing the changes (both the API addition and code cleanup). 
This would be split into 2 PRs if this API addition is accepted.

https://github.com/apache/trafficserver/pull/8313

-Randall

Reply via email to