Re: pgarchives: strip angle brackets when checking for msgid search - Mailing list pgsql-www

From Magnus Hagander
Subject Re: pgarchives: strip angle brackets when checking for msgid search
Date
Msg-id CABUevEw-VKrQm3M7zLOmwBeRjp+MgafxHKzE0zgaDM65PyrsCA@mail.gmail.com
Whole thread Raw
In response to pgarchives: strip angle brackets when checking for msgid search  (Amir Rohan <amir.rohan@zoho.com>)
Responses Re: pgarchives: strip angle brackets when checking for msgid search
Archive viewer mangles links to archive appearing in messages
List pgsql-www
On Fri, Oct 2, 2015 at 7:29 PM, Amir Rohan <amir.rohan@zoho.com> wrote:
In some email-clients, when you copy the Message-ID to the clipboard
it surrounds in (or perhaps, doesn't strip) angle brackets.
When pasting this into the mail archive search, search fails to find the message
and this happens often enough so as to be annoying. The fix is trivial.

Discussion of similar problem in the commitfest app:

<CABUevExRGr+Q5gzp-9cDLOqgntsOEcM0zZYg5VkADj3Z_NKz5A@mail.gmail.com>

patch attached.

Interesting - we also had a similar fix in 9b7e9b59 for the archives, but that one only deals with the URLs and not the search box. Makes much sense to do it in the search box as well, so applied. Thanks! 

Actually, before I sent this. It's also broken because it redirects to bad URLs if there are spaces in it. Or if there is just one of the two, e.g. it has < but not >. All those now generate 404s, which is not good.

This patch needs to be in sync with the callers of that API as well as the code that actually views entries in the archives before it can be safely applied.

--

pgsql-www by date:

Previous
From: Magnus Hagander
Date:
Subject: Re: Deselecting "Receive Mail" on the website ml subscribe form is broken
Next
From: Amir Rohan
Date:
Subject: Re: pgarchives: strip angle brackets when checking for msgid search