Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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 CreditAttribution: 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 CreditAttribution: erikwebb commentedsubscribe
Comment #3
jaarong CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: erikwebb commentedComment #7
webkenny CreditAttribution: webkenny commentedThe real difference was the protocol order. To keep it consistent, I have alphabetized that (moving
tel
beforetelnet
) - 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.