zwoop edited a comment on pull request #7516: URL: https://github.com/apache/trafficserver/pull/7516#issuecomment-810503823
> I'm also curious on how this information is intended to be used. If the intention is for mainly exposing the TCP-INFO to access logs, I wonder if it makes sense to consider using log tags instead of header-rewrite operator? Yeh, primarily for logs. Moving it to the core is an option, but the thought was to avoid polluting the core more with a bunch of #ifdef's (and also makes it more difficult for people to make custom additions or changes). But if the consensus is that it'd be better to move this to the core, rather than a plugin, I'm fine with that too. Also, and this might be complicated, is that you may very likely want to have multiple TCPinfo log fields, for different hooks. Like, captured at the beginning of a session, and then at the end of each TXN. -- 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]
