From: | Paul Ohlhauser <bendix(dot)ohlhauser(at)gmail(dot)com> |
---|---|
To: | postgresql(dot)wizard(at)gmail(dot)com |
Cc: | pgsql-hackers(at)lists(dot)postgresql(dot)org |
Subject: | Re: [PG19-3 PATCH] Don't ignore passfile |
Date: | 2025-09-04 08:59:36 |
Message-ID: | CAGbOXJGL1r=T7-CEByR1PCk24MV4=1j6Ajj0Bsg1WDVc5TgdjQ@mail.gmail.com |
Views: | Whole Thread | Raw Message | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Hi Umar,
thanks for taking a look and pointing to the documentation.
While it is good that this behavior is noted in the documentation, the
issues mentioned in my proposal still stand:
- The warning itself does not indicate that the "passfile" is ignored. Of
course users can and hopefully will eventually look at the documentation,
but the behavior is not self-explanatory nor intuitive (even though it
could be). Reality is, that most users do not read the documentation for
each directive they use to the very end. As libpg is used in many other
software products, users consulting the Postgres documentation specifically
is less common. I really don't want to defend the user potentially being
negligent - I am saying that the current behavior has no/minimal use and
could be more intuitive to spare users time.
- While the documentation addresses one or two peripheral issues I
mentioned, it does not address the fundamental issue, that the current
behavior is not useful / in the users interest.
- Not being able to use group permissions is also still a prohibitive
constraint.
I'll also include the gist of the patch here as it is a 2-3 line change:
- fclose(fp);
- return NULL;
Kind regards
Paul Ohlhauser
PS: I changed my email address in the mailing list, hope that's not an
issue.
From | Date | Subject | |
---|---|---|---|
Next Message | jian he | 2025-09-04 09:00:02 | Re: pg_restore --no-policies should not restore policies' comment |
Previous Message | Nataliia | 2025-09-04 08:40:47 | Re: Timeline switching with partial WAL records can break replica recovery |