Document spaces in .pgpass need to be escaped

Started by James Colemanover 4 years ago3 messageshackers
Jump to latest
#1James Coleman
jtc331@gmail.com

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

Attachments:

v1-0001-Document-spaces-in-.pgpass-need-to-be-escaped.patchapplication/octet-stream; name=v1-0001-Document-spaces-in-.pgpass-need-to-be-escaped.patchDownload+3-3
#2Tom Lane
tgl@sss.pgh.pa.us
In reply to: James Coleman (#1)
Re: Document spaces in .pgpass need to be escaped

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

#3James Coleman
jtc331@gmail.com
In reply to: Tom Lane (#2)
Re: Document spaces in .pgpass need to be escaped

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