WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
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
Details
View All
Add attachment
proposed patch, testcase, etc.
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.
Top of Page
Format For Printing
XML
Clone This Bug