Thread: Document spaces in .pgpass need to be escaped

Document spaces in .pgpass need to be escaped

From
James Coleman
Date:
A coworker has a space in a Postgres password and noticed .pgpass
didn't work; escaping it fixed the issue. That requirement wasn't
documented (despite other escaping requirements being documented), so
I've attached a patch to add that comment.

Thanks,
James Coleman

Attachment

Re: Document spaces in .pgpass need to be escaped

From
Tom Lane
Date:
James Coleman <jtc331@gmail.com> writes:
> A coworker has a space in a Postgres password and noticed .pgpass
> didn't work; escaping it fixed the issue. That requirement wasn't
> documented (despite other escaping requirements being documented), so
> I've attached a patch to add that comment.

I looked at passwordFromFile() and I don't see any indication that
it treats spaces specially.  Nor does a quick test here confirm
this report.  So I'm pretty certain that this proposed doc change
is wrong.  Perhaps there's some other issue to investigate, though?

            regards, tom lane



Re: Document spaces in .pgpass need to be escaped

From
James Coleman
Date:
On Wed, Sep 29, 2021 at 12:13 PM Tom Lane <tgl@sss.pgh.pa.us> wrote:
>
> James Coleman <jtc331@gmail.com> writes:
> > A coworker has a space in a Postgres password and noticed .pgpass
> > didn't work; escaping it fixed the issue. That requirement wasn't
> > documented (despite other escaping requirements being documented), so
> > I've attached a patch to add that comment.
>
> I looked at passwordFromFile() and I don't see any indication that
> it treats spaces specially.  Nor does a quick test here confirm
> this report.  So I'm pretty certain that this proposed doc change
> is wrong.  Perhaps there's some other issue to investigate, though?
>
>                         regards, tom lane

Thanks for taking a look.

I'm honestly not sure what happened here. I couldn't reproduce again
either, and on another box with this coworker we could reproduce, but
then realized the pgpass entry was missing a field. I imagine it must
have been similar on the original box we observed the error on, but
both of our memories were of just adding teh escape characters...

I'll mark the CF entry as withdrawn.

Thanks,
James Coleman