Closed
Bug 1503493
Opened 7 years ago
Closed 7 years ago
Add validation to the URL field
Categories
(bugzilla.mozilla.org :: Bug Creation/Editing, enhancement)
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: kohei, Assigned: kohei)
References
Details
Split from Bug 1436619. Currently the URL field doesn’t have any validation. I think it should have.
* It has to be loose, since we have accepted `data:`, `javascript:` and any other values on the URL field.
* Frontend: use `<input type="url">`
* Backend: use a library; introduce `FIELD_TYPE_URL`
|
Assignee | |
Comment 1•7 years ago
|
||
WIP, which is actually an earlier version of the Bug 1436619 work:
https://github.com/kyoshino/bmo/compare/bug-1503493-validate-url
Assignee: nobody → kohei.yoshino
Status: NEW → ASSIGNED
|
|
Assignee | |
Comment 2•7 years ago
|
||
A quick search finds 232 open bugs w/ an invalid URL:
https://bugzilla.mozilla.org/buglist.cgi?bug_file_loc=%5E%5B%5E%28http%7Cftp%7Cjavascript%7Cdata%7Cnews%29%5D&bug_file_loc_type=regexp&resolution=---&query_format=advanced
And 147 of them start with one or more spaces:
https://bugzilla.mozilla.org/buglist.cgi?bug_file_loc=%5E%5C%20%2B%5Ba-z%5D&bug_file_loc_type=regexp&resolution=---&query_format=advanced
|
|
Assignee | |
Comment 3•7 years ago
|
||
Given the number of invalid URLs, I’m closing this as WONTFIX for now.
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•