jasmine-nahrain commented on code in PR #12391:
URL: https://github.com/apache/trafficserver/pull/12391#discussion_r2230127416
##########
plugins/experimental/cache_fill/background_fetch.cc:
##########
@@ -176,6 +208,12 @@ BgFetchData::initialize(TSMBuffer request, TSMLoc req_hdr,
TSHttpTxn txnp)
if (set_header(mbuf, hdr_loc, TS_MIME_FIELD_HOST,
TS_MIME_LEN_HOST, hostp, len)) {
Dbg(dbg_ctl, "Set header Host: %.*s", len, hostp);
}
+ // Next, remove the Range headers and IMS (conditional) headers
from the request
Review Comment:
Without removing them, only the range requested will be cached. If we remove
them, then the whole body will be cached. This is for the background_fetch
request.
--
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: github-unsubscr...@trafficserver.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
