Re: [PATCH] pgweb: auth.py: make it possible to customize email address in error message

From: Magnus Hagander <magnus(at)hagander(dot)net>
To: Célestin Matte <celestin(dot)matte(at)cmatte(dot)me>
Cc: pgsql-www(at)postgresql(dot)org
Subject: Re: [PATCH] pgweb: auth.py: make it possible to customize email address in error message
Date: 2025-12-01 20:14:11
Message-ID: CABUevEwTbnzYycjxUG+biuZ7Zxcf4yeYLPOx876VPk7Aqcji3w@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-www

On Thu, 21 Dec 2023 at 18:39, Célestin Matte <celestin(dot)matte(at)cmatte(dot)me>
wrote:

> > Maybe the correct choice is to actually add it to pglister?
>
> Alright, patched for pglister attached.
> Also attached a patch for pgarchives, because while ADMINS is defined,
> there is no default value. (Note: this variable is only used for auth.py)
>

Once again sorry to be ridiculously late :)

Realizing that this auth.py code is shared along basically all
postgresql.org properties, I think the conditional way is the right way to
do it. I don't think we should prepopulate a value of
webmaster(at)postgresql(dot)org into the ADMIN values in the configs -- but we
should add an empty one. so I've pushed that change for now.

//Magnus

In response to

Browse pgsql-www by date

  From Date Subject
Next Message Magnus Hagander 2025-12-01 20:30:00 Re: Support for listing the beta versions
Previous Message Magnus Hagander 2025-12-01 20:00:47 Re: pglister: pgarchives does not synchronize subscribers with email delivery disabled (proposed patch)