Re: Improving psql's \password command

From: "Bossart, Nathan" <bossartn(at)amazon(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: "pgsql-hackers(at)lists(dot)postgresql(dot)org" <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Improving psql's \password command
Date: 2021-11-12 17:23:58
Message-ID: E0C46CDF-46B2-4388-B53B-96EB2CA0F71E@amazon.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 11/11/21, 8:12 AM, "Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> It seems to me it'd overcomplicate simple_prompt's API for one use-case
> ... but if you want to try it, step right up. (I suppose some of that
> objection could be overcome by making simple_prompt into a wrapper
> around another function not_so_simple_prompt.)

I haven't started on the SIGINT stuff yet, but I did extract the
PQuser() fix so that we can hopefully get that one out of the way. I
made some small adjustments in an attempt to keep the branching in
this function from getting too complicated.

Nathan

Attachment Content-Type Size
v2-0001-Adjust-password-to-use-current-user-instead-of-au.patch application/octet-stream 1.7 KB
v2-0002-Add-role-name-to-password-prompt.patch application/octet-stream 1.7 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Justin Pryzby 2021-11-12 17:31:02 Re: Parallel Append can break run-time partition pruning
Previous Message Fabrice Chapuis 2021-11-12 16:57:41 Re: Logical replication timeout problem