Project

General

Profile

Bug #15976

rgw: updating CORS/ACLs might not work in some circumstances

Added by Radoslaw Zarzynski almost 3 years ago. Updated over 2 years ago.

Status:
Resolved
Priority:
High
Target version:
-
Start date:
05/21/2016
Due date:
% Done:

0%

Source:
other
Tags:
Backport:
jewel
Regression:
Yes
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:

Description

This is because std::map::emplace(), that replaced operator[] in metadata processing code, doesn't override already existing items.


Related issues

Duplicated by rgw - Bug #16439: Can not change access for containers Duplicate 06/23/2016
Duplicated by rgw - Bug #16694: Radosgw use swift API create container ACL failed Duplicate 07/15/2016
Copied to rgw - Backport #16040: jewel: rgw: updating CORS/ACLs might not work in some circumstances Resolved

History

#1 Updated by Radoslaw Zarzynski almost 3 years ago

  • Status changed from New to In Progress

#2 Updated by Radoslaw Zarzynski almost 3 years ago

  • Related to Bug #15779: rgw: account/container metadata not actually present in a request are deleted during POST through Swift API added

#3 Updated by Radoslaw Zarzynski almost 3 years ago

  • Related to deleted (Bug #15779: rgw: account/container metadata not actually present in a request are deleted during POST through Swift API)

#4 Updated by Radoslaw Zarzynski almost 3 years ago

  • Status changed from In Progress to Need Review

#5 Updated by Radoslaw Zarzynski almost 3 years ago

  • Status changed from Need Review to Pending Backport

#6 Updated by Nathan Cutler almost 3 years ago

  • Copied to Backport #16040: jewel: rgw: updating CORS/ACLs might not work in some circumstances added

#7 Updated by tingting ma almost 3 years ago

Before excuting the function of emplace_attr. remove corresponding item, which already exists,
After modifying code,it runs correctly.

#8 Updated by Radoslaw Zarzynski over 2 years ago

  • Duplicated by Bug #16439: Can not change access for containers added

#9 Updated by Radoslaw Zarzynski over 2 years ago

  • Duplicated by Bug #16694: Radosgw use swift API create container ACL failed added

#10 Updated by Radoslaw Zarzynski over 2 years ago

  • Status changed from Pending Backport to Resolved

Also available in: Atom PDF