Code review comment for lp:~ted/url-dispatcher/more-sophisticated-urls

Revision history for this message
Olivier Tilloy (osomon) wrote :

It seems that "tel:///" is not correct indeed. Although I couldn’t find proper anecdotal evidence, this question on stackoverflow (http://stackoverflow.com/questions/22704662/using-slashes-in-tel-uris) seems to imply that there are tel links in the wild with two slashes (which is definitely incorrect).

As I understand it, the generic RE is used for matching "http" URLs as well as "tel", so unless we introduce a more specific RE for matching "tel" URLs, the generic RE will accept "tel" links with slashes, right?

I don’t think that’s a big deal though, the issue at hand (handling "tel:" links) is fixed, the logic can probably be refined later with a lower priority.

« Back to merge proposal