Hi all,

I'd like to propose a change to an existing TS API, "TSHttpTxnAborted".

The current function signature:
TSReturnCode
TSHttpTxnAborted(TSHttpTxn txnp)

Propsed change:
TSReturnCode
TSHttpTxnAborted(TSHttpTxn txnp, bool *client_abort)

Currently, the purpose of the API is limited to providing success/error on 
overall transaction aborts. This change addresses a particular use case 
(recently encountered at LinkedIn), where we want to distinguish client-side 
aborts from the actual server aborts. While client aborts may not be indicative 
of any direct problems with downstream servers, it's the server aborts that can 
be used as passive health-checks for the downstream origin hosts. Also, this 
makes troubleshooting the 'transaction aborts' easier because now we are able 
to eliminate an entire set of either client-side or server-side investigations, 
due to the ability to distinguish between them.

Related PR - https://github.com/apache/trafficserver/pull/7901/files
Related gh issue - https://github.com/apache/trafficserver/issues/7902

Please provide your feedback and let me know if more information is needed from 
my side.

Thanks,

Sanjay

Reply via email to