#15229 closed (fixed)
URLValidator class to accept ftp links
Reported by: | codefisher | Owned by: | nobody |
---|---|---|---|
Component: | Core (Other) | Version: | dev |
Severity: | Keywords: | ||
Cc: | Triage Stage: | Accepted | |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description
The URLValidator class currently only accepts links that are http or https. I would suggest that maybe adding ftp as well would be valuable. I have attached a patch.
Attachments (2)
Change History (6)
by , 14 years ago
comment:1 by , 14 years ago
Needs tests: | set |
---|---|
Patch needs improvement: | set |
Triage Stage: | Unreviewed → Accepted |
by , 14 years ago
Attachment: | 15229.patch added |
---|
comment:4 by , 14 years ago
I just noticed that I should have updated the comment (http, https, ftp, or ftps).
Note:
See TracTickets
for help on using tickets.
patch including tests for https, ftp, and ftps urls