I don't think I'd do it for TLS_CLOSE, but VCONN_CLOSE would likely be worth it. There's an issue on that, linked from the issue I mentioned above. It's about the same thing - plugin wants to attach data during TLS handshake for later retrieval at the HTTP level that can be cleaned up if something goes wrong.
On Wed, Sep 27, 2017 at 4:29 PM, Dk Jack <dnj0...@gmail.com> wrote: > Yeah, I was thinking along similar lines... > > Do you think, it'd be a worth while effort to add a ssl-close event hook? > > On Wed, Sep 27, 2017 at 1:31 PM, Alan Carroll < > solidwallofc...@oath.com.invalid> wrote: > > > I'd probably put a time stamp in and update it during the hook > processing. > > If you waited 5 minutes or so before clearing, that should be enough. If > > it's been 5 minutes since the last activity the connection should be > dead, > > unless you're shipping big files. > > > > Or, alternatively, in READ_REQUEST_HDR_HOOK pull the data out and attach > it > > to the txn, cleaning it up in TXN_CLOSE. Set a time stamp on entry in the > > table and if that's been 5 minutes, the connection has definitely timed > > out. > > >
