| 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
| 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) |