RESOLVED DUPLICATE of bug 230935 168433
[GTK] Lack of support for CrossOriginPreflight XHR request
https://bugs.webkit.org/show_bug.cgi?id=168433
Summary [GTK] Lack of support for CrossOriginPreflight XHR request
Javier Fernandez
Reported 2017-02-16 07:11:39 PST
The change in r204795 added support on WebCore for redirect post CORS-preflight requests. This enabled to follow cross-origin redirections for not-simple requests, through preflight checks. We were not processing the non-simple request on http/tests/xmlhttprequest/redirections-and-user-headers.html fails because of this, but now they are being processed. I don t think such test fails because of that, but it definitively triggered some missing features of the GTK port.
Attachments
Current diff of the test failure. (9.80 KB, text/html)
2017-02-16 08:43 PST, Javier Fernandez
no flags
Javier Fernandez
Comment 1 2017-02-16 07:12:28 PST
Please, Youenn, if you have time take a look.
youenn fablet
Comment 2 2017-02-16 08:19:30 PST
This code should be cross-platform but hey, we never know! Looking at the current GTK result, it does not seem related to CORS/preflight since one same-origin redirection test is not passing either (Authentication header present). Is r204795 regressing the test in GTK?
Javier Fernandez
Comment 3 2017-02-16 08:39:03 PST
(In reply to comment #2) > This code should be cross-platform but hey, we never know! > Looking at the current GTK result, it does not seem related to > CORS/preflight since one same-origin redirection test is not passing either > (Authentication header present). > > Is r204795 regressing the test in GTK? Not exactly. The test was already failing before r204795 but with a different error. I think the new errors come from the fact that after r204795 we allow same-origin redirections for non-simple requests. My request for review would be more to get some clue about what webkitgtk could be missing to cause the current failure of this test.
Javier Fernandez
Comment 4 2017-02-16 08:43:00 PST
Created attachment 301771 [details] Current diff of the test failure.
Arcady Goldmints-Orlov
Comment 5 2021-11-19 14:31:48 PST
This appears to have been fixed under 230935. *** This bug has been marked as a duplicate of bug 230935 ***
Note You need to log in before you can comment on or make changes to this bug.