LoCo Team Portal

false URLs in event description.

Bug #734232 reported by YoBoY
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
LoCo Team Portal
Won't Fix
Low
Unassigned

Bug Description

When you enter some text containing a possible url finishing by .net .com or .org (possibly more) the text before the extension is clickable. I don't have this effect when it's finishing by a two letter extension (tested with .fr, .pt, .uk).

You can test with (don't work if there is no text before) :
* test.net
* realy/complicated.test/with_lot/of/impossibility.com
* (pourquoi.jetombe/toujours.surdes.bug/bizarre.com)

The urls should always start with http:// or https:// to be valid

One event where I can see the bug : http://loco.ubuntu.com/events/team/804/detail/

Revision history for this message
Michael Hall (mhall119) wrote :

These links are created by Django's built-in urlize template filter, which I believe only supports .com .net and .org tlds. Maybe we should look into 3rd party filters or writing out own to cover more tlds.

I disagree that the http/https should be required, often people post websites just starting with www. or just wiki.ubuntu.com and we would want to linkify those too.

Revision history for this message
YoBoY (yoboy-leguesh) wrote :

You are right, limiting to the http:// and https:// is not the solution, but we have at least to limit the letters before the .com .org .net and not allow forbiden characters in the host.

Dajngo urlize filter : http://docs.djangoproject.com/en/dev/ref/templates/builtins/?from=olddocs#urlize

This bug should be reported to django also if someone can do that.

Revision history for this message
Adnane Belmadiaf (daker) wrote :

The filter only works on http, https, www. links and links ending in .org, .net or .com, maybe we can just added the missing cases to it http://code.djangoproject.com/browser/django/trunk/django/utils/html.py#L102

Chris Johnston (cjohnston)
Changed in loco-directory:
status: New → Confirmed
importance: Undecided → Low
Revision history for this message
Adnane Belmadiaf (daker) wrote :

Marking this as "Won't fix" since we use the markdown syntax.

Changed in loco-team-portal:
status: Confirmed → Won't Fix
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.