Re: [PATCH] pglister: auth_receive: Indicate when PGAUTH_KEY is invalid instead of crashing

Поиск
Список
Период
Сортировка
От Magnus Hagander
Тема Re: [PATCH] pglister: auth_receive: Indicate when PGAUTH_KEY is invalid instead of crashing
Дата
Msg-id CABUevEyc_XzoUA7ML0pz37-AdR4vd16FzFeNxCtPb3=Wh3OFuw@mail.gmail.com
обсуждение исходный текст
Ответ на Re: [PATCH] pglister: auth_receive: Indicate when PGAUTH_KEY is invalid instead of crashing  (Célestin Matte <celestin.matte@cmatte.me>)
Ответы Re: [PATCH] pglister: auth_receive: Indicate when PGAUTH_KEY is invalid instead of crashing
Список pgsql-www
On Mon, Feb 20, 2023 at 7:36 PM Célestin Matte <celestin.matte@cmatte.me> wrote:
>
> Update: attached patched also catch ValueError exceptions, which can happen as well.
>
> Also attaching similar patch for pgarchives.
>
> As being confused by the lack of error message happened to me again, I think these patches are useful.

It seems a patch like this was already merged into the upstream of
this, and got deployed into pgarchives & pglister when I synced those
up a couple of weeks ago. Sorry about the lateness of that one. Does
the current version of the code work for what you were looking for?

It might also be worth considering also checking explicitly for the
format of PGAUTH_KEY on startup. Something like the attached maybe?
(against pgweb which is the master of this file) What do you think?

--
 Magnus Hagander
 Me: https://www.hagander.net/
 Work: https://www.redpill-linpro.com/

Вложения

В списке pgsql-www по дате отправления:

Предыдущее
От: Magnus Hagander
Дата:
Сообщение: Re: pgarchives: Bug report + Patches: loader can't handle message in multiple lists
Следующее
От: Palak Chaturvedi
Дата:
Сообщение: Editor access to wiki