| From: | Andreas Karlsson <andreas(at)proxel(dot)se> |
|---|---|
| To: | Kuroda, Hayato/黒田 隼人 <kuroda(dot)hayato(at)fujitsu(dot)com>, "'pgsql-hackers(at)lists(dot)postgresql(dot)org'" <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
| Subject: | Re: [doc] pg_ctl: fix wrong description for -l |
| Date: | 2026-04-03 23:15:59 |
| Message-ID: | 8b587a55-148f-4cb4-8941-8fed282625a9@proxel.se |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On 3/23/26 11:55 AM, Kuroda, Hayato/黒田 隼人 wrote:
> ```
> -l filename
> --log=filename
> Append the server log output to filename. If the file does not exist, it is created. The umask is set to 077, so access to the log file is disallowed to other users by default.
> ```
>
> But users who are in the same group can also see the log if group permissions are
> enabled on the data directory. It was extended by c37b3d0 but updating the doc seemed
> to be missed. Attached patch can fix the issue.
Thanks for the patch.
Nicely spotted, but I am not sure I am a fan of the new text. Shouldn't
we still mention that "If the file does not exist, it is created."? Also
I think the last sentence would flow better as:
Access to the log file is disallowed to other users by default, but
users in the same group can also read it if group permissions are
enabled on the data directory.
Best regards,
Andreas
--
Andreas Karlsson
Percona
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Jacob Champion | 2026-04-03 23:26:42 | Re: unclear OAuth error message |
| Previous Message | Andres Freund | 2026-04-03 23:10:48 | Re: AIO / read stream heuristics adjustments for index prefetching |