hassanakbar4 / mailarchive-tickets

0 stars 0 forks source link

Linkification broken for angle bracket enclosed URLs #308

Open hassanakbar4 opened 3 years ago

hassanakbar4 commented 3 years ago

component_MailArchive: User Interface resolution_ type_defect | by rcross@amsl.com


Hello! I have spotted a problem with URL linkification in the mail archive: it looks like URLs are not linkified inside RFC 3986 angle bracket delimiters https://tools.ietf..org/html/rfc3986#appendix-C

Example message showing the problem:

https://mailarchive.ietf.org/arch/msg/dhcwg/pMkq7ZS98PsQMIKUSUuUYAFILHw/

In the quoted paragraph starting “1.” a URL is linkified, then in the immediately following text there is another URL which is not linkified.


Issue migrated from trac:3102 at 2021-09-22 16:57:22 +0500

hassanakbar4 commented 3 years ago

@hassanakbar4 commented


This looks to be a problem with the order of message text processing. First the text is escaped which results in this:

<https://mailarchive.ietf.org/arch/msg/dhcwg/5o2w-njCCxFJp1Omkiaal7KFRSw&gt;

This foils the Django template filter urlize which otherwise would properly handle angle bracket enclosed URLs.

Need to correct the order of processing without breaking something else.

hassanakbar4 commented 3 years ago

@hassanakbar4 changed status from new to closed

hassanakbar4 commented 3 years ago

@hassanakbar4 changed resolution from ` tofixed`

hassanakbar4 commented 3 years ago

@hassanakbar4 commented


Fixed in [1050/mailarch]:

Fix linkification of angle brace enclosed urls. Fixes #3102

hassanakbar4 commented 3 years ago

@hassanakbar4 changed status from closed to reopened

hassanakbar4 commented 3 years ago

@hassanakbar4 changed resolution from fixed to ``

hassanakbar4 commented 3 years ago

@hassanakbar4 commented


The fix for this issue caused other issues with text/html MIME types and had to be backed out.

This message for example, https://mailarchive.ietf.org/arch/msg/ietf/qv-H-b8zNQZAiq2YGD9xs83DjBo/, was displayed escaped. "

..."

hassanakbar4 commented 3 years ago

@hassanakbar4 changed _comment0 which not transferred by trac-hub