Re: [doc] pg_ctl: fix wrong description for -l

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

In response to

Browse pgsql-hackers by date

  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