Re: [PATCH pglister] Add Archived-At header to delivered messages

Hi,

Magnus Hagander a écrit :
> On Fri, Jan 29, 2021 at 4:38 PM Denis Laxalde <denis(dot)laxalde(at)dalibo(dot)com> wrote:
>> Notes:
>> To build the link, I used the urlpattern field from archive_server table
>> and replace '/list/' by '/message-id/':
>>
>> https://www.postgresql.org/list/% -> https://www.postgresql.org/message-id/
>>
>> The urlpattern field is now retrieved in 'mailinglists' view, hence the
>> migration.
>>
>> That's not very elegant, but I did not find a better way to achieve this.
>> Perhaps adding a 'messageidpattern' column to 'lists_archiveserver' table
>> would be better? Or maybe I missed something?
>
> There is already a field called "mailurlpattern" that links to the
> individual message. Was there a particular reason you couldn't use
> that one, or did you just miss that it existed?

I just missed that it existed; thanks for pointing that out.
Please find attached an updated version of the patch.

Denis