Strip leading and trailing C0 controls and space for urls passed to new URL()

RESOLVED FIXED in Firefox 49

Status

()

RESOLVED FIXED
3 years ago
2 years ago

People

(Reporter: 446240525, Assigned: valentin)

Tracking

unspecified
mozilla49
Points:
---

Firefox Tracking Flags

(firefox49 fixed)

Details

(Whiteboard: [necko-active] btpp-active)

Attachments

(1 attachment)

(Reporter)

Description

3 years ago
>new URL("\0http://mozilla.org\1")
TypeError:  is not a valid URL // \0 and \1 should be stripped first

spec: https://url.spec.whatwg.org/#concept-basic-url-parser

Comment 1

3 years ago
Seems like we might call into the wrong entry point for the URL parser?
(Assignee)

Updated

3 years ago
Assignee: nobody → valentin.gosu
Whiteboard: [necko-active]
Whiteboard: [necko-active] → [necko-active] btpp-active
(Assignee)

Comment 5

2 years ago
Created attachment 8754426 [details] [diff] [review]
Strip leading and trailing C0 controls and space for urls passed to new URL()

MozReview-Commit-ID: B7BiKE6P7GA
Attachment #8754426 - Flags: review?(mcmanus)
Comment on attachment 8754426 [details] [diff] [review]
Strip leading and trailing C0 controls and space for urls passed to new URL()

Review of attachment 8754426 [details] [diff] [review]:
-----------------------------------------------------------------

thanks for taking this on
Attachment #8754426 - Flags: review?(mcmanus) → review+
(Assignee)

Comment 7

2 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/a10683e0ed001963fcf87a55d42b9e189fd4f481
Bug 1271955 - Strip leading and trailing C0 controls and space for urls passed to new URL() r=mcmanus

Comment 9

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/a10683e0ed00
https://hg.mozilla.org/mozilla-central/rev/08111f812323
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox49: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla49
You need to log in before you can comment on or make changes to this bug.