Opened 8 years ago

Closed 7 years ago

#6279 closed (fixed)

urlize doesn't escape greater-than and less-than signs surrounding email address

Reported by: ishirav@… Owned by: nobody
Component: Template system Version: master
Severity: Keywords: urlize
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:

Description

When you give urlize (with autoescape) an email address surrounding by greater-than and less-than signs like this:

<kuku@gmail.com>

it converts the mail address to a link, but doesn't escape the surrounding greater-than and less-than signs.

Attachments (1)

urlize.diff (2.3 KB) - added by Rob Hudson <treborhudson@…> 8 years ago.
Adding a patch and regression test

Download all attachments as: .zip

Change History (3)

Changed 8 years ago by Rob Hudson <treborhudson@…>

Adding a patch and regression test

comment:1 Changed 7 years ago by Simon Greenhill <dev@…>

  • Has patch set
  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset
  • Triage Stage changed from Unreviewed to Ready for checkin

comment:2 Changed 7 years ago by mtredinnick

  • Resolution set to fixed
  • Status changed from new to closed

(In [7079]) Fixed #6279, #6514 -- Fixed some HTML escaping problems in the urlize filter.
Based on a patch from SmileyChris with some test additions from Rob Hudson.
Thanks, both.

Note: See TracTickets for help on using tickets.
Back to Top