Project

General

Profile

Bug #6078

rgw: CORS not working

Added by Yehuda Sadeh over 10 years ago. Updated over 10 years ago.

Status:
Resolved
Priority:
Urgent
Assignee:
Target version:
-
% Done:

0%

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

History

#1 Updated by Yehuda Sadeh over 10 years ago

  • Backport set to dumpling

#2 Updated by Yehuda Sadeh over 10 years ago

  • Status changed from 12 to Fix Under Review

Pushed a bunch of changes to wip-6078.

#3 Updated by Yehuda Sadeh over 10 years ago

  • Assignee changed from Yehuda Sadeh to Josh Durgin

#4 Updated by Josh Durgin over 10 years ago

  • Assignee changed from Josh Durgin to Yehuda Sadeh

Left a couple comments on github. It'd be nice to try to break up some of the changes and explicitly explain why changes are made more in future commits. I.e. https://github.com/ceph/ceph/commit/7bdcc79124504f3d924714ddfa0d60feb7c7209d could be split into method extraction, reordering error checks (I can infer why, but it won't be so clear without context months from now), and finally adding generic cors response.

#5 Updated by Yehuda Sadeh over 10 years ago

  • Status changed from Fix Under Review to Pending Backport

#6 Updated by Yehuda Sadeh over 10 years ago

  • Status changed from Pending Backport to Resolved

Done. Backported to dumpling, merged at commit:a304016fa01b02efd500135c00b9bf3407a9999c. Also, commit:670db7e80ddc9c26c43a4f66907a5996ce207c4d.

Also available in: Atom PDF