On Tue, Dec 17, 2013 at 12:17:23AM -0800, Yehuda Sadeh wrote: > On Sun, Dec 15, 2013 at 1:16 PM, Robin H. Johnson <robbat2@xxxxxxxxxx> wrote: > > Hi, > > > > I was trying to use the RGW CORS support, and found that it flubbed > > handling of the Access-Control-Request-Headers header badly. > > > > It was expecting Access-Control-Allow-Headers in the request where it > > should have taken the Access-Control-Request-Headers, and it was also > > validating the content in a case-sensitive manner, when the CORS > > standard (6.2.6) requires that headers are validated case-insensitive. > > > > I have submitted a pull request with the fixes: > > https://github.com/ceph/ceph/pull/949 > > > > I think they should be backported to emperor as well. > Overall it looks ok, I still want to run the test suite against it. > Can you open corresponding issues in the ceph tracker? > (tracker.ceph.com). http://tracker.ceph.com/issues/7043 Pull request recreated because I relocated the repo (for work reasons): https://github.com/ceph/ceph/pull/975 I've also got an unrelated fix I'm working on in the S3 POST code, I think the signature validation is wrong. -- Robin Hugh Johnson Gentoo Linux: Developer, Infrastructure Lead E-Mail : robbat2@xxxxxxxxxx GnuPG FP : 11ACBA4F 4778E3F6 E4EDF38E B27B944E 34884E85 -- To unsubscribe from this list: send the line "unsubscribe ceph-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
