ywkaras commented on a change in pull request #6800:
URL: https://github.com/apache/trafficserver/pull/6800#discussion_r427404964
##########
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:
But doesn't this mean you could not be sure that your override of
handleTxnClose() would actually get run? Because the continuation that runs
handleTransactionEvents() may run first. I don't think we ever merged the
proposed continuation priority changes (@duke8253 @SolidWallOfCode ?). I
think thats what you'd need for this to be useful.
----------------------------------------------------------------
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]
