Make WordPress Core

Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#21081 closed enhancement (fixed)

Whitelist the tel: and fax: URL protocols.

Reported by: markjaquith's profile markjaquith Owned by: markjaquith's profile markjaquith
Milestone: 3.5 Priority: low
Severity: minor Version: 3.4
Component: General Keywords: has-patch 2nd-opinion
Focuses: Cc:

Description

We should support telephone and fax number URLs, by whitelisting their protocols: tel: and fax:

http://www.ietf.org/rfc/rfc2806.txt

e.g.

tel:2025551234

Attachments (2)

21081.diff (1.3 KB) - added by markjaquith 12 years ago.
21081.2.diff (1.4 KB) - added by markjaquith 12 years ago.

Download all attachments as: .zip

Change History (9)

@markjaquith
12 years ago

#1 @markjaquith
12 years ago

  • Keywords has-patch 2nd-opinion added; needs-patch removed
  • Owner set to markjaquith
  • Status changed from new to accepted

#2 @ocean90
12 years ago

I thought we had added the filter for adding more protocols, see #18268.

#3 follow-up: @markjaquith
12 years ago

We added a filter. Doesn't mean we shouldn't add some more defaults. Telephone links are particularly handy for local businesses creating mobile sites.

#4 @nacin
12 years ago

You have +1 from me.

@markjaquith
12 years ago

#5 @markjaquith
12 years ago

.2 patch makes the edit_user() call use wp_allowed_protocols(), so there are no other places to keep having to update this protocol list.

#6 @markjaquith
12 years ago

  • Resolution set to fixed
  • Status changed from accepted to closed

In [21170]:

Allow tel: and fax: protocols. Wrangle the last hardcoded protocol enumeration so that it uses wp_allowed_protocols(). fixes #21081

#7 in reply to: ↑ 3 @ocean90
12 years ago

Replying to markjaquith:

We added a filter. Doesn't mean we shouldn't add some more defaults. Telephone links are particularly handy for local businesses creating mobile sites.

Yeah, of course. I just saw the changed line in wp-admin/includes/user.php which doesn't used wp_allowed_protocols. But now it does and I'm happy.

Note: See TracTickets for help on using tickets.