Project

General

Profile

Actions
Copy link

Bug #64439

open

If-Match Header Not Unquoted On Conditional PutObject

Added by Damian Peckett 3 months ago. Updated 3 months ago.

Status:
New
Priority:
Normal
Assignee:
Ali Maredia
Target version:
Ceph - v19.0.0
% Done:

0%

Source:
Community (dev)
Tags:
Backport:
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

Looks like some codepaths have been missed over the years and the s3tests seem to be supplying an unquoted ETag.

I have a proposed fix on a branch here: https://github.com/dpeckett/ceph/commit/0ae123dafef3274a1f3ab292915554cbe9701ce6

Let me know if anything is missing from the commit, the s3tests probably also need to be corrected.

Actions
Copy link
 #1

Updated by Casey Bodley 3 months ago

thanks for the commit!
looking at https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/If-Match i do see ETag values in quotes, but the * is not quoted. so i think the comparison against * should happen before rgw_string_unquote()

Actions
Copy link
 #2

Updated by Ali Maredia 3 months ago

  • Assignee set to Ali Maredia
Actions
Copy link

Also available in: Atom PDF