public inbox for pgsql-www@postgresql.org  

help / color / mirror / Atom feed

From: Magnus Hagander <magnus@hagander.net>
To: Célestin Matte <contact@cmatte.me>
Cc: PostgreSQL WWW <pgsql-www@lists.postgresql.org>
Subject: Re: pglister: pgarchives does not synchronize subscribers with email delivery disabled (proposed patch)
Date: Mon, 1 Dec 2025 21:00:47 +0100
Message-ID: <CABUevEyGraEg-z_602j8oukikOYy05j-Raxgt8Ndi3WGsi069Q@mail.gmail.com> (raw)
In-Reply-To: <b39abaaf-00f0-47b9-b819-3b22d83a67d8@cmatte.me>
References: <b39abaaf-00f0-47b9-b819-3b22d83a67d8@cmatte.me>

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

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-www@postgresql.org
  Cc: magnus@hagander.net, contact@cmatte.me, pgsql-www@lists.postgresql.org
  Subject: Re: pglister: pgarchives does not synchronize subscribers with email delivery disabled (proposed patch)
  In-Reply-To: <CABUevEyGraEg-z_602j8oukikOYy05j-Raxgt8Ndi3WGsi069Q@mail.gmail.com>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox