Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.The 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 attributes.
The RFC for the Telephone protocol is located here: http://www.ietf.org/rfc/rfc2806.txt
Might be time to consider this. A backport of this issue is located here for interested D6 parties: http://drupal.org/node/931304#comment-3590058
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | tel-protocol-addition-d7.patch | 919 bytes | webkenny |
| tel-protocol-D7.patch | 694 bytes | webkenny |
Comments
Comment #1
webkenny commentedSetting to needs review for the bot.
Comment #3
webkenny commentedMy patching skills a little rusty these days apparently. Here's a clean one with the necessary re-roll to keep the array contents alpha sorted. The reason the last patch failed the test was my use of diff to create it. :)
Comment #4
moshe weitzman commentedComment #5
dries commentedCommitted to CVS HEAD. Thanks.
Comment #6
David_Rothstein commentedDoesn't this introduce an inconsistency with http://api.drupal.org/api/function/_filter_url/7, which uses the same variable? It seems like that one should be updated too...
(Maybe a separate followup issue; it really seems like this list should be stored in a single helper function rather than a couple different places. That would prevent it from easily getting out of sync like this.)
Comment #8
David_Rothstein commentedA few years later, the followup actually did happen :) #2026269: Different default values for filter_allowed_protocols variable
Comment #9
webkenny commentedWe really know how to make the words, "Never too late" mean something around here. :) Thanks for the post, David.