zwoop commented on PR #9480: URL: https://github.com/apache/trafficserver/pull/9480#issuecomment-1458744264
This is actually worse than I first expected, there's about 230 calls to this function. However, upon inspecting this, the problem is that ``` class HttpAPIHooks : public FeatureAPIHooks<TSHttpHookID, TS_HTTP_LAST_HOOK> ``` Creates (give or take) 180 or so APIHook objects which can never be used. There's a big gap between the last HTTP Hook (TS_HTTP_REQUEST_CLIENT_HOOK) and the first SSL hook (TS_SSL_FIRST_HOOK=201). I'm thinking that we need to move the last HTTP hook to be something like ``` TS_HTTP_LAST_HOOK = TS_HTTP_REQUEST_CLIENT_HOOK, ``` -- 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]
