Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.I'm not sure why, but when I do something like
<a href:"tel:555.555.5555">555.555.5555</a>
the tel: part of href is filtered out when run through filter_xss_admin. This is being used more and more on mobile sites and I would like to be able to output this through Views.
| Comment | File | Size | Author |
|---|---|---|---|
| #7 | d6-tel-protocol-v2.patch | 1 KB | webkenny |
| #1 | tel-protocol-D6.patch | 806 bytes | webkenny |
Comments
Comment #1
webkenny commentedThe tel protocol is growing in its use on mobile devices. The attached patch adds it to the allowed list of protocols. Thanks goes to erikwebb for helping me locate the source of the stripped attribute.
Quick edit. The RFC for the Telephone protocol is located here: http://www.ietf.org/rfc/rfc2806.txt
Comment #2
erikwebb commentedsubscribe
Comment #3
jaarong commentedI've tested this and it is working as expected. Also, for those that might be confused, I typed the code wrong in the first post, should be
<a href="tel:555.555.5555">555.555.5555</a>Comment #4
gábor hojtsyLooks like a slightly different version of the patch was committed to D7 at #945332: Allow "tel:" protocol through drupal_strip_dangerous_protocols. Is this ready to be committed then?
Comment #5
erikwebb commentedLooks innocuous to me and an identical patch was committed to D7 HEAD yesterday.
Patch applies cleanly and the simple added functionality is there without side effects.
Comment #6
erikwebb commentedComment #7
webkenny commentedThe real difference was the protocol order. To keep it consistent, I have alphabetized that (moving
telbeforetelnet) - Also, having not used CVS to create the original patch (Bad Kenny!) I ended up inadvertently changing the file name. Proper re-roll attached. Thanks for the testing.Leaving this RTBC since the change was alphabetical.
Comment #9
gábor hojtsySuperb, committed, thank you.