bryancall commented on issue #5718: URL: https://github.com/apache/trafficserver/issues/5718#issuecomment-4835268124
I dug into this while triaging the backlog. Part of what this issue asks for has landed, but I do not think it is fully resolved, so I am leaving it open. [PR #5769](https://github.com/apache/trafficserver/pull/5769) "Simplify h2 enable disable" (shipped in 9.0.0) removed the expensive per-connection clone path: it deleted `SSLNextProtocolSet::clone()`, `advertiseProtocols()`, `unregisterEndpoint()`, and the cached `npn` buffer, and switched the per-connection enable/disable to a `SessionProtocolSet` bit set (`enableProtocol`/`disableProtocol` flip bits, and the ALPN advertisement is now built on the fly by testing `enabled.contains(index)`). Important caveat though: `SessionProtocolSet` already existed as a bit set before this issue was filed, so #5769 adopted an existing bit set for one path rather than doing the broader conversion described here. The wider ask, unifying the three configuration routes (port configuration, `sni.yaml`, and the plugin APIs) onto bit-fields, with predefined sets per protocol and room for HTTP/3 and QUIC, does not appear to be fully done. Leaving this open for a maintainer to decide whether #5769 covers enough to close, or whether the remaining unification is still wanted. -- 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]
