masaori335 commented on code in PR #13158:
URL: https://github.com/apache/trafficserver/pull/13158#discussion_r3245281579
##########
src/proxy/http/HttpSM.cc:
##########
@@ -5091,6 +5091,35 @@ HttpSM::do_range_setup_if_necessary()
if (t_state.cache_info.action == HttpTransact::CacheAction_t::REPLACE)
{
if (t_state.hdr_info.server_response.status_get() == HTTPStatus::OK)
{
Dbg(dbg_ctl_http_range, "Serving transform after stale cache
re-serve");
+
+ // Ranges and range_output_cl were computed against the stale
cached object size. If the fresh origin Content-Length
+ // differs, re-parse the Range against the fresh value so the
outgoing Content-Length/Content-Range match the body
+ // actually being sent. Without this, Content-Length/Content-Range
advertise the stale cached size.
+ const int64_t fresh_cl =
t_state.hdr_info.server_response.get_content_length();
+ const int64_t cached_cl = t_state.cache_info.object_read ?
t_state.cache_info.object_read->object_size_get() : -1;
+ if (fresh_cl > 0 && fresh_cl != cached_cl) {
+ SMDbg(dbg_ctl_http_range, "Re-parsing range against fresh origin
Content-Length %" PRId64 " (was %" PRId64 ")",
+ fresh_cl, cached_cl);
+ delete[] t_state.ranges;
+ t_state.ranges = nullptr;
+ t_state.num_range_fields = 0;
+ t_state.range_setup = HttpTransact::RangeSetup_t::NONE;
+ t_state.range_output_cl = 0;
+ parse_range_done = false;
+
+ std::string_view content_type =
+
t_state.hdr_info.server_response.value_get(static_cast<std::string_view>(MIME_FIELD_CONTENT_TYPE));
+ parse_range_and_compare(field, fresh_cl);
+ calculate_output_cl(content_type.length(),
num_chars_for_int(fresh_cl));
+
+ if (t_state.range_setup !=
HttpTransact::RangeSetup_t::REQUESTED) {
+ // Re-parse yielded e.g. RANGE_NOT_SATISFIABLE (entire range
+ // past fresh body); let downstream handling take over without
+ // installing the transform.
Review Comment:
Add a test cases in AuTest to cover this scenario.
--
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]
