Skip to content

squid: rgw/s3: remove 'aws-chunked' from Content-Encoding response#65219

Merged
yuriw merged 1 commit into
ceph:squidfrom
cbodley:wip-72715-squid
Sep 18, 2025
Merged

squid: rgw/s3: remove 'aws-chunked' from Content-Encoding response#65219
yuriw merged 1 commit into
ceph:squidfrom
cbodley:wip-72715-squid

Conversation

@cbodley
Copy link
Copy Markdown
Contributor

@cbodley cbodley commented Aug 25, 2025

backport tracker: https://tracker.ceph.com/issues/72715


backport of #64440
parent tracker: https://tracker.ceph.com/issues/21128

this backport was staged using ceph-backport.sh version 16.0.0.6848
find the latest version at https://github.com/ceph/ceph/blob/main/src/script/ceph-backport.sh

PutObject stores some of the generic http request headers in object
attrs so they can be returned as response headers in Get/HeadObject

S3 has its own `aws-chunked` value for the `Content-Encoding` header,
which it says does _not_ get stored with the object or returned with
Get/HeadObject

we've been storing this header with objects forever, so omitting the
value on PutObject doesn't fix the issue for existing objects. instead,
add the necessary filtering to Get/HeadObject

Fixes: https://tracker.ceph.com/issues/21128

Signed-off-by: Casey Bodley <cbodley@redhat.com>
(cherry picked from commit 4a802b8)
@cbodley cbodley requested a review from a team as a code owner August 25, 2025 16:24
@cbodley cbodley added this to the squid milestone Aug 25, 2025
@yuriw yuriw merged commit dc393c9 into ceph:squid Sep 18, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants