bneradt commented on code in PR #13154:
URL: https://github.com/apache/trafficserver/pull/13154#discussion_r3243827634
##########
include/proxy/ProxyTransaction.h:
##########
@@ -146,18 +146,19 @@ class ProxyTransaction : public VConnection
void mark_as_tunnel_endpoint() override;
- /** Emit a best-effort access log entry for a request that failed before
- * HttpSM creation.
+ /** Emit a best-effort access log entry for a request without an HttpSM.
*
* Call this when a malformed request is rejected at the protocol layer
* (e.g. during HTTP/2 or HTTP/3 header decoding) and no HttpSM was
- * created. The method populates a PreTransactionLogData from the
+ * created. The method populates a NonHttpSmLogData from the
* session and the partially decoded request, then invokes Log::access.
+ * If an HttpSM exists, callers should use the normal transaction logging
+ * path instead.
*
* @param[in] request The decoded (possibly partial) request header.
* @param[in] protocol_str Protocol string for the log entry (e.g. "http/2").
*/
- void log_pre_transaction_access(HTTPHdr const *request, const char
*protocol_str);
+ void log_non_http_sm_access(HTTPHdr const *request, const char
*protocol_str);
Review Comment:
log_pre_transaction_access / log_non_http_sm_access are not externally
visible so this is not a concern.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
