Bug #6078
rgw: CORS not working
% 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.