Re: pgsql: Add 'basebackup_to_shell' contrib module. - Mailing list pgsql-hackers

From Tom Lane
Subject Re: pgsql: Add 'basebackup_to_shell' contrib module.
Date
Msg-id 648071.1648745113@sss.pgh.pa.us
Whole thread Raw
In response to Re: pgsql: Add 'basebackup_to_shell' contrib module.  (Robert Haas <robertmhaas@gmail.com>)
Responses Re: pgsql: Add 'basebackup_to_shell' contrib module.  (Robert Haas <robertmhaas@gmail.com>)
Re: pgsql: Add 'basebackup_to_shell' contrib module.  (Andrew Dunstan <andrew@dunslane.net>)
List pgsql-hackers
Robert Haas <robertmhaas@gmail.com> writes:
> On Thu, Mar 31, 2022 at 12:25 PM Andrew Dunstan <andrew@dunslane.net> wrote:
>> Yeah, I think the fix is as simple as the attached.

> Well, that does not work because you added an extra parenthesis which
> makes Perl barf. If you fix that, then the test does not pass because,
> as I just explained to Tom, the flag we call --create-role doesn't
> create a role:

> error running SQL: 'psql:<stdin>:1: ERROR:  role "backupuser" does not exist'

On looking closer, the combination of --config-auth and --create-role
*only* fixes the config files for SSPI, it doesn't expect the server
to be running.

I agree that the documentation of this is nonexistent and the design
is probably questionable, but I'm not volunteering to fix either.
If you are, step right up.  In the meantime, I believe (without
having tested) that the correct incantation is to use auth_extra
but *also* create the user further down.

            regards, tom lane



pgsql-hackers by date:

Previous
From: Fabrice Fontaine
Date:
Subject: [PATCH] src/interfaces/libpq/Makefile: fix pkg-config without openssl
Next
From: Fabien COELHO
Date:
Subject: Re: psql - add SHOW_ALL_RESULTS option