|From:||Julian Markwort <julian(dot)markwort(at)uni-muenster(dot)de>|
|To:||Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr>|
|Cc:||PostgreSQL Developers <pgsql-hackers(at)postgresql(dot)org>|
|Subject:||Re: [PATCH] pgpassfile connection option|
|Views:||Raw Message | Whole Thread | Download mbox | Resend email|
On 10/16/2016 12:09 PM, Fabien COELHO wrote:
> Patch applies cleanly, make check ok... however AFAICS it only means
> that it compiles but it is not tested in anyway... This is is
> annoying. Well I'm not sure whether other options are tested either,
> but they should.
Thanks for taking the time to review my patch! I haven't found much
regarding the testing of connection options. However since there isn't
anything fancy going on in PasswordFromFile() I'm not too worried about
> The documentation needs to be updated.
I've written a couple lines now.
I aligned the definition of the connection option and the environment
variable with that of other (conn.opt&env.var.) pairs and added mention
of the different options to the doc of the "Password File".
> The reported password file is wrong. It is even more funny if
> ~/.pgpass contains the right password: the pgpassfile option *is*
> taken into account, ./foo is read and it fails, but the error message
> is not updated and points to the wrong file. The error message stuff
> should be updated to look for the pgpassfile connection string option...
That was indeed an Error on my side, I hadn't updated the errormessages
to inform you which file has been used.
So attached is an updated version of the patch.
I'd like to ask for some input on how to handle invalid files - right
now no message is shown, the user just gets a password prompt as a
result, however I think a message when the custom pgpassfile hasn't been
found would be helpful.
Westphalian Wilhelms-University in Münster
|Next Message||Noah Misch||2016-10-25 14:57:24||Recommend wrappers of PG_DETOAST_DATUM_PACKED()|
|Previous Message||Michael Paquier||2016-10-25 13:35:16||Re: Quorum commit for multiple synchronous replication.|