Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WebKit export of https://bugs.webkit.org/show_bug.cgi?id=169851 #5192

Merged
merged 1 commit into from Mar 22, 2017

Conversation

youennf
Copy link
Contributor

@youennf youennf commented Mar 22, 2017

Safari sends empty "Access-Control-Request-Headers" in preflight request


This change is Reviewable

@wpt-pr-bot
Copy link
Collaborator

Notifying @annevk and @jdm. (Learn how reviewing works.)

@w3c-bots
Copy link

View the complete job log.

Firefox (nightly channel)

Testing web-platform-tests at revision c77e39e
Using browser at version BuildID 20170320110254; SourceStamp 1b9293be51637f841275541d8991314ca56561a5
Starting 10 test iterations
All results were stable

All results

2 tests ran
/fetch/api/cors/cors-preflight-worker.html
Subtest Results Messages
OK
CORS [DELETE], server allows PASS {}
CORS [DELETE], server refuses PASS {}
CORS [PUT], server allows PASS {}
CORS [PUT], server refuses PASS {}
CORS [PATCH], server allows PASS {}
CORS [PATCH], server refuses PASS {}
CORS [NEW], server allows PASS {}
CORS [NEW], server refuses PASS {}
CORS [GET] [x-test-header: allowed], server allows PASS {}
CORS [GET] [x-test-header: refused], server refuses PASS {}
CORS [GET] [several headers], server allows PASS {}
CORS [GET] [several headers], server refuses PASS {}
CORS [PUT] [several headers], server allows PASS {}
CORS [PUT] [several headers], server refuses PASS {}
CORS [PUT] [only safe headers], server allows PASS {}
/fetch/api/cors/cors-preflight.html
Subtest Results Messages
OK
CORS [DELETE], server allows PASS
CORS [DELETE], server refuses PASS
CORS [PUT], server allows PASS
CORS [PUT], server refuses PASS
CORS [PATCH], server allows PASS
CORS [PATCH], server refuses PASS
CORS [NEW], server allows PASS
CORS [NEW], server refuses PASS
CORS [GET] [x-test-header: allowed], server allows PASS
CORS [GET] [x-test-header: refused], server refuses PASS
CORS [GET] [several headers], server allows PASS
CORS [GET] [several headers], server refuses PASS
CORS [PUT] [several headers], server allows PASS
CORS [PUT] [several headers], server refuses PASS
CORS [PUT] [only safe headers], server allows PASS

@w3c-bots
Copy link

View the complete job log.

Chrome (unstable channel)

Testing web-platform-tests at revision c77e39e
Using browser at version 59.0.3047.0 dev
Starting 10 test iterations
All results were stable

All results

2 tests ran
/fetch/api/cors/cors-preflight-worker.html
Subtest Results Messages
OK
CORS [DELETE], server allows PASS {}
CORS [DELETE], server refuses PASS {}
CORS [PUT], server allows PASS {}
CORS [PUT], server refuses PASS {}
CORS [PATCH], server allows PASS {}
CORS [PATCH], server refuses PASS {}
CORS [NEW], server allows PASS {}
CORS [NEW], server refuses PASS {}
CORS [GET] [x-test-header: allowed], server allows PASS {}
CORS [GET] [x-test-header: refused], server refuses PASS {}
CORS [GET] [several headers], server allows PASS {}
CORS [GET] [several headers], server refuses PASS {}
CORS [PUT] [several headers], server allows PASS {}
CORS [PUT] [several headers], server refuses PASS {}
CORS [PUT] [only safe headers], server allows PASS {}
/fetch/api/cors/cors-preflight.html
Subtest Results Messages
OK
CORS [DELETE], server allows PASS
CORS [DELETE], server refuses PASS
CORS [PUT], server allows PASS
CORS [PUT], server refuses PASS
CORS [PATCH], server allows PASS
CORS [PATCH], server refuses PASS
CORS [NEW], server allows PASS
CORS [NEW], server refuses PASS
CORS [GET] [x-test-header: allowed], server allows PASS
CORS [GET] [x-test-header: refused], server refuses PASS
CORS [GET] [several headers], server allows PASS
CORS [GET] [several headers], server refuses PASS
CORS [PUT] [several headers], server allows PASS
CORS [PUT] [several headers], server refuses PASS
CORS [PUT] [only safe headers], server allows PASS

@annevk
Copy link
Member

annevk commented Mar 22, 2017

This looks identical to #5191?

@youennf
Copy link
Contributor Author

youennf commented Mar 22, 2017

My mistake, should be good now.

@annevk annevk merged commit 631c0d4 into web-platform-tests:master Mar 22, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants