Bug 220783

Summary:

Form payloads with text/plain enctype should have a final \r\n

Product:

WebKit

Reporter:

Andreu Botella <andreu>

Component:

Forms

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

akeerthi, andreu, cdumez, darin, ews-watchlist, japhet, smoley, webkit-bug-importer, wenson_hsieh

Priority:

Keywords:

InRadar

Version:

WebKit Nightly Build

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Andreu Botella

Reported 2021-01-20 17:16:38 PST

Text/plain form payloads have entries separated by a \r\n newline, but the HTML specification (https://html.spec.whatwg.org/#plain-text-form-data) requires there to be a final \r\n newline, which WebKit skips. Tests: https://wpt.fyi/results/html/semantics/forms/form-submission-0/text-plain.window.html?label=pr_head&max-count=1&pr=26740 (ignore the "formdata event" tests, since those are meant to test entries added to the form entry list through the formdata event, which WebKit doesn't yet implement)

Attachments
Patch (18.66 KB, patch) 2021-11-14 11:02 PST, Andreu Botella	no flags	Details Formatted Diff Diff
Patch (18.77 KB, patch) 2021-11-14 19:28 PST, Andreu Botella	no flags	Details Formatted Diff Diff
Patch (20.17 KB, patch) 2021-11-15 12:25 PST, Andreu Botella	no flags	Details Formatted Diff Diff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.

Radar WebKit Bug Importer

Comment 1 2021-01-21 16:45:16 PST

Andreu Botella

Comment 2 2021-11-14 11:02:57 PST

Andreu Botella

Comment 3 2021-11-14 19:28:43 PST

Andreu Botella

Comment 4 2021-11-15 12:25:12 PST

EWS

Comment 5 2021-11-29 10:15:19 PST

Committed r286221 (244602@main): <https://commits.webkit.org/244602@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 444292 [details].

Note You need to log in before you can comment on or make changes to this bug.