sudheerv commented on a change in pull request #6800:
URL: https://github.com/apache/trafficserver/pull/6800#discussion_r427413533
##########
File path: src/tscpp/api/utils_internal.cc
##########
@@ -141,6 +155,15 @@ void inline invokePluginForEvent(Plugin *plugin, TSHttpTxn
ats_txn_handle, TSEve
case TS_EVENT_HTTP_CACHE_LOOKUP_COMPLETE:
plugin->handleReadCacheLookupComplete(transaction);
break;
+ case TS_EVENT_HTTP_TXN_CLOSE:
+ if (plugin) {
Review comment:
No, the continuation for my plugin will still run regardless of
`handleTransactionEvents()`. That's how the stack trace above resulted. It's
like registering CPPAPI twice with the same hook.
`handleTransactionEvents()` will have no affect on TransactionPlugins() that
never called addPlugin()
----------------------------------------------------------------
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]
