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
Support form submission of multipart/form-data #10712
Conversation
Heads up! This PR modifies the following files:
|
b70b651
to
9b35e2c
Compare
@bors-servo try |
Support form submission of multipart/form-data Fixes #7553. <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/10712) <!-- Reviewable:end -->
💔 Test failed - mac-rel-wpt |
9b35e2c
to
2852a48
Compare
Looks like no WPT tests changed results; what's the test plan here? |
I'm very surprised that no new WPT tests pass, I'm guessing I must be doing something wrong in this PR. |
Ah wait, it's because there aren't any consumers of the functionality yet. |
☔ The latest upstream changes (presumably #10750) made this pull request unmergeable. Please resolve the merge conflicts. |
I'm tempted to also implement submit as body action for forms just to make tests for this, but this'll have to wait until saturday or later since I have final exams and I have some moving to do. |
2852a48
to
6b21d02
Compare
6b21d02
to
755fcde
Compare
a2b362b
to
62cb7c1
Compare
if request.body == "foo=bara": | ||
return [("Content-Type", "text/plain")], "OK" | ||
else: | ||
return [("Content-Type", "text/plain")], "FAIL" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This doesn't work right now - it keeps on returning "Unknown content type (application/json)" to the iframe. Not sure what's wrong here...
62cb7c1
to
04fac3a
Compare
Review status: 0 of 11 files reviewed at latest revision, 3 unresolved discussions. tests/wpt/web-platform-tests/html/semantics/forms/form-submission-0/submit-entity-body.html, line 21 [r5] (raw file): tests/wpt/web-platform-tests/html/semantics/forms/form-submission-0/submit-entity-body.html, line 39 [r5] (raw file): Comments from Reviewable |
☔ The latest upstream changes (presumably #10833) made this pull request unmergeable. Please resolve the merge conflicts. |
3105c6a
to
e9f6b83
Compare
Support form submission of multipart/form-data Fixes #7553. <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/10712) <!-- Reviewable:end -->
💔 Test failed - linux-dev |
538e24c
to
3110647
Compare
@bors-servo r=jdm |
📌 Commit 3110647 has been approved by |
⌛ Testing commit 3110647 with merge 77cb2ca... |
Support form submission of multipart/form-data Fixes #7553. <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/10712) <!-- Reviewable:end -->
💔 Test failed - mac-rel-wpt |
|
⚡ Previous build results for android, arm32, arm64, linux-dev, linux-rel, mac-dev-unit, mac-rel-css are reusable. Rebuilding only mac-rel-wpt, windows... |
☀️ Test successful - android, arm32, arm64, linux-dev, linux-rel, mac-dev-unit, mac-rel-css, mac-rel-wpt, windows |
Fixes #7553.
This change is