Opened 10 years ago
Closed 10 years ago
#37300 closed enhancement (fixed)
Add 'urn' to allowed protocols
| Reported by: | geekysoft | Owned by: | ocean90 |
|---|---|---|---|
| Priority: | normal | Milestone: | 4.7 |
| Component: | General | Version: | |
| Severity: | normal | Keywords: | has-patch |
| Cc: | Focuses: |
Description
'urn' should be the safest protocol to allow on the planet as it doesn’t do much, except act as an identifier in various XML schemes.
the_guid() currently uses esc_url() as a default filter, meaning urn:uuid:<hash> can’t be used as a guid in feeds ithout first allowing the urn protocol. The Atom feed standard shows most examples with a urn:uuid used as the ID. (IDs in Atom must be URIs so using esc_url() still makes perfect sense.)
The attach patch adds the urn protocol to the list of allowed protocols. (See RFC 2141 for details on the protocol schema.)
Work-around: print( get_the_guid() ); doesn’t apply any default filters like the_guid() does.
Attachments (4)
Change History (10)
#1
@
10 years ago
- Keywords has-patch added
- Milestone Awaiting Review → Future Release
- Type defect (bug) → enhancement
- Version trunk
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
@geekysoft Thanks for the ticket and the patch, looks good to me. The function needs only a new changelog entry for the new protocol, see [32346] for 'webcal'.