Make WordPress Core

Opened 3 years ago

Last modified 2 months ago

#39415 new enhancement

Adding sms to list of allowed protocol

Reported by: rilwis Owned by:
Milestone: 5.3 Priority: normal
Severity: normal Version: 4.8
Component: Formatting Keywords: has-patch dev-feedback
Focuses: Cc:


Currently, the list of allowed protocols is provided in the function wp_allowed_protocols() . However, the list doesn’t have the "sms" protocol.

This is a new protocol and very useful for mobile users as they can send SMS when clicking the link. It’s very similar to "tel" protocol where users can call a person when clicking the link.



Attachments (1)

39415.patch (1.7 KB) - added by keraweb 6 months ago.
Added 'skype' and 'sms' to the protocols array.

Download all attachments as: .zip

Change History (6)

#1 @agusmu
3 years ago

personally, I like this idea.

the problems are:
(1) sms protocol is very buggy
(2) it has different format for different devices (android, iphone, old iphone iOS7/8, etc)
(3) it doesn't work on facebook in-app browser

#2 @rilwis
3 years ago

Thanks agusmu for your comment.

I think the format of the protocol is not a problem. All of them use the same "sms:" protocol, the only difference is the following text. I just mean we add the protocol, not for supporting the links.

If we add the protocol, the devices which support that (iPhone, Android) will be able to open the SMS/Phone app and do its job. Think about "skype:" protocol, it works only if your computer installed Skype.

This ticket is about supporting the protocol, not making it work. Making it work is the totally different story and in my opinion, is out of the scope of WordPress itself.

6 months ago

Added 'skype' and 'sms' to the protocols array.

#3 @keraweb
6 months ago

  • Keywords has-patch dev-feedback added

I agree with @rilwis. Whether the end user can use it is not up to WordPress. WordPress should IMO allow site owners to choose themselves if they want to use these protocols or not. I've also added skype since that wasn't in there as well.

#4 @SergeyBiryukov
2 months ago

#46579 was marked as a duplicate.

#5 @SergeyBiryukov
2 months ago

  • Component changed from General to Formatting
  • Milestone changed from Awaiting Review to 5.3
Note: See TracTickets for help on using tickets.