Re: Improving psql's \password command

Поиск
Список
Период
Сортировка
От Bossart, Nathan
Тема Re: Improving psql's \password command
Дата
Msg-id 9D3BFA94-DEF2-4B43-B240-8ADC595ECA7A@amazon.com
обсуждение исходный текст
Ответ на Re: Improving psql's \password command  (Tom Lane <tgl@sss.pgh.pa.us>)
Ответы Re: Improving psql's \password command  (Tom Lane <tgl@sss.pgh.pa.us>)
Список pgsql-hackers
On 11/19/21, 9:17 AM, "Tom Lane" <tgl@sss.pgh.pa.us> wrote:
> "Bossart, Nathan" <bossartn@amazon.com> writes:
>> Okay, here is an attempt at adding control-C support for \prompt and
>> \connect.  It was reasonably straightforward.  I did have to teach
>> simple_prompt_extended() to add a newline after a cancellation when
>> "echo" is true.
>
> LGTM, pushed after very minor fooling with the comments.

Thanks!

>> I think that's it for psql.  After a quick glance, I didn't see any
>> other obvious candidates for control-C support, but I'll look a little
>> closer to be sure.
>
> Hmm, initdb's prompt-for-superuser-password might need it.
> I think the rest of our frontend programs don't trap SIGINT,
> or at least don't do so while requesting user input.

I'm able to cancel the superuser password prompt in initdb already.
It looks like the signal handlers aren't set up until after
get_su_pwd().  I did find some missing control-C handling in
pg_receivewal/pg_recvlogical, though.  Attached is a patch for those.

Nathan


Вложения

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

Предыдущее
От: Andres Freund
Дата:
Сообщение: Re: Adding CI to our tree
Следующее
От: "Bossart, Nathan"
Дата:
Сообщение: Re: Should rename "startup process" to something else?