Re: pgarchives: strip angle brackets when checking for msgid search

From: Amir Rohan <amir(dot)rohan(at)zoho(dot)com>
To: Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>
Cc: Magnus Hagander <magnus(at)hagander(dot)net>, PostgreSQL www <pgsql-www(at)postgresql(dot)org>
Subject: Re: pgarchives: strip angle brackets when checking for msgid search
Date: 2015-10-05 14:38:20
Message-ID: 56128B5C.4020308@zoho.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-www

On 10/05/2015 04:59 PM, Alvaro Herrera wrote:
> Amir Rohan wrote:
>
>> With these patches (now touching pgweb and pgarchives, when a (cleaned)
>> search query gets a hit on a msgid we now include the canonical
>> messageid in the JSON response. I didn't have it return a url as you
>> suggested because archives should know as little as possible
>> about pgweb's link structure. So pgweb still crafts the redirect url,
>> only it uses the returned msgid if its present in the response.
>
> FWIW the URL structure for archives can never be changed, because many
> URLs are in use in permanent storage such as commit messages and
> archived emails. As an example, when we migrated from the old
> Mhonarc-generated urls (with the mbox/month/message number structure)
> we, or Magnus actually, created a database with redirects from that
> structure to the message-id-based URL structure, to avoid breaking
> links.
>
> I'm not saying your approach is bad or shouldn't be used, just pointing
> out something that the rest of the -www team is probably already aware
> of and you might not be.
>

That's a valid point (two, actually) but it still /logically/
belongs in the the main website instead of the archive "API server",
and keeps with how this worked before.

But if magnus concurs with you, It's easily changed.

Amir

In response to

Browse pgsql-www by date

  From Date Subject
Next Message Amir Rohan 2015-10-05 18:05:55 Re: No easy way to join discussion in existing thread when not subscribed
Previous Message Alvaro Herrera 2015-10-05 13:59:24 Re: pgarchives: strip angle brackets when checking for msgid search