| Summary: | [GTK] http/tests/misc/form-submit-file-cross-site-redirect.html fails with incomplete form data | ||||||
|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Lauro Moura <lmoura> | ||||
| Component: | Tools / Tests | Assignee: | Lauro Moura <lmoura> | ||||
| Status: | RESOLVED FIXED | ||||||
| Severity: | Normal | CC: | bugs-noreply, clopez, commit-queue, webkit-bug-importer | ||||
| Priority: | P2 | Keywords: | InRadar | ||||
| Version: | Other | ||||||
| Hardware: | Unspecified | ||||||
| OS: | Unspecified | ||||||
| Attachments: |
|
||||||
(In reply to Lauro Moura from comment #0) > With b206517 fixed tips: - instead of " b206517 " use format: " bug 206517 " so bugzilla makes automatic links - if the bug its related to GTK or WPE releated, always CC the virtual user bugs-noreply@webkitgtk.org Created attachment 390796 [details]
Patch
The commit-queue encountered the following flaky tests while processing attachment 390796 [details]: editing/spelling/spellcheck-async-remove-frame.html bug 158401 (authors: morrita@google.com, rniwa@webkit.org, and tony@chromium.org) The commit-queue is continuing to process your patch. Comment on attachment 390796 [details] Patch Clearing flags on attachment: 390796 Committed r256638: <https://trac.webkit.org/changeset/256638> All reviewed patches have been landed. Closing bug. |
With b206517 fixed, this test submits the test file metadata correctly (like the non-redirect version) but no payload is sent. --- /home/lauro/dev/WebKit/layout-test-results/http/tests/misc/form-submit-file-cross-site-redirect-expected.txt +++ /home/lauro/dev/WebKit/layout-test-results/http/tests/misc/form-submit-file-cross-site-redirect-actual.txt @@ -2,6 +2,6 @@ This tests verifies the test file included in the multipart form data: PASS: File is present PASS: File upload was successful -PASS: File size is greater than 0 +FAIL: File size is 0 Payload with `var_dump($_FILES['data']);` in the PHP script: ``` array(5) { ["name"]=> string(8) "test.tmp" ["type"]=> string(24) "application/octet-stream" ["tmp_name"]=> string(14) "/tmp/php0dQ3qM" ["error"]=> int(0) ["size"]=> int(0) } ``` Using the original test from Julio in b201950[1] with Minibrowser it seems to work normally. [1] https://redirection-experiment.herokuapp.com/