Actions
Backport #15283
closedhammer: RGW shouldn't send Content-Type nor Content-Length for 304 responses
Release:
hammer
Pull request ID:
Crash signature (v1):
Crash signature (v2):
Updated by Nathan Cutler about 8 years ago
- Copied from Bug #14005: RGW shouldn't send Content-Type nor Content-Length for 304 responses added
Updated by Wei-Chung Cheng about 8 years ago
- Description updated (diff)
- Status changed from New to In Progress
Updated by Sylvain Munaut about 8 years ago
The current pull request for backport :
https://github.com/ceph/ceph/pull/8379/commits/e53751d0e52f6fb611d9bd81fe2115cd7ce533b3
Is definitely not good. err will be equal to -ERR_NOT_MODIFIED (note the minus sign).
Not sure if the same comment applies to the original (non-backport) patch. I only use hammer ATM.
Updated by Orit Wasserman almost 8 years ago
it is also wrong in master, do you want to send a fix?
Updated by Nathan Cutler almost 8 years ago
Orit Wasserman wrote:
it is also wrong in master, do you want to send a fix?
Opened #16327 for this bug, master PR is https://github.com/ceph/ceph/pull/9725
Wei-Chung Cheng, when the master PR is merged, could you cherry-pick fc38346 into this backport PR?
Updated by Nathan Cutler almost 8 years ago
- Related to Bug #16327: comparing return code to ERR_NOT_MODIFIED in rgw_rest_s3.cc (needs minus sign) added
Updated by Nathan Cutler almost 8 years ago
https://github.com/ceph/ceph/pull/9725 was merged.
Updated by Nathan Cutler almost 8 years ago
- Related to deleted (Bug #16327: comparing return code to ERR_NOT_MODIFIED in rgw_rest_s3.cc (needs minus sign))
Updated by Nathan Cutler almost 8 years ago
- Related to Backport #16382: hammer: comparing return code to ERR_NOT_MODIFIED in rgw_rest_s3.cc (needs minus sign) added
Updated by Nathan Cutler over 7 years ago
- Status changed from In Progress to Resolved
- Target version set to v0.94.8
Actions