bneradt commented on code in PR #12667:
URL: https://github.com/apache/trafficserver/pull/12667#discussion_r2557690282
##########
include/proxy/http/HttpTransact.h:
##########
@@ -635,6 +635,7 @@ class HttpTransact
bool trust_response_cl = false;
ResponseError_t response_error =
ResponseError_t::NO_RESPONSE_HEADER_ERROR;
bool extension_method = false;
+ std::string server_response_transfer_encoding; ///< Storage for
logging.
Review Comment:
Thanks for the review.
current.server->transfer_encoding is an enum over these values:
```
enum class TransferEncoding_t {
NONE = 0,
CHUNKED,
DEFLATE,
};
```
That will not allow us to fully replicate the original string, which might
contain "deflate", "chunked", or the like and any combination of them. The ops
users of this field want to know the full string sent from the origin, but we
delete the field entirely. Thus it is always currently empty. This PR stores
the origin resonse's string for logging.
--
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]
