Re: pglister: pgarchives does not synchronize subscribers with email delivery disabled (proposed patch) - Mailing list pgsql-www

From Magnus Hagander
Subject Re: pglister: pgarchives does not synchronize subscribers with email delivery disabled (proposed patch)
Date
Msg-id CABUevEyGraEg-z_602j8oukikOYy05j-Raxgt8Ndi3WGsi069Q@mail.gmail.com
Whole thread Raw
In response to pglister: pgarchives does not synchronize subscribers with email delivery disabled (proposed patch)  (Célestin Matte <contact@cmatte.me>)
List pgsql-www


On Mon, 22 Sept 2025 at 20:18, Célestin Matte <contact@cmatte.me> wrote:
The pglister_sync script is supposed to synchronize subscribers from pglister to pgarchives, so they can access archives.
 From what I understand of its semantics, the "nomail" field (labeled "Disable mail delivery") is used to remain subscribed, to be able to access archives, without receiving emails.
However, due to the way the API works, this is not the case. pglister_sync.py makes a call to the API, which get subscribers from the mailinglist_subscribers SQL view. This view is built in the 0027_merge_migrations.py migrations and excludes users with nomail.
I propose a patch with a migration to change this SQL view to include nomail users (copying previous definition, without "AND NOT nomail").
Note that this changes results of the API, which might have consequences if other scripts use it (I haven't found any with some grepping).

Hi!

mailinglist_subscribers is used for actual mail delivery as well, isn't it? If you just change the view, it will change all deliveries and basically the nomail switch to not work at all?

While you're on the right track to include them, don't we also need to include the nomail column as a column in the view and then filter on that when sending email?

//Magnus

pgsql-www by date:

Previous
From: Magnus Hagander
Date:
Subject: Re: [PATCH] pgarchives: bugfix: increase listsubscription username length
Next
From: Magnus Hagander
Date:
Subject: Re: [PATCH] pgweb: auth.py: make it possible to customize email address in error message