Re: [doc] pg_ctl: fix wrong description for -l - Mailing list pgsql-hackers

From Andreas Karlsson
Subject Re: [doc] pg_ctl: fix wrong description for -l
Date
Msg-id 8b587a55-148f-4cb4-8941-8fed282625a9@proxel.se
Whole thread Raw
In response to [doc] pg_ctl: fix wrong description for -l  (Kuroda, Hayato/黒田 隼人 <kuroda.hayato@fujitsu.com>)
List 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
accessto 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




pgsql-hackers by date:

Previous
From: Andres Freund
Date:
Subject: Re: AIO / read stream heuristics adjustments for index prefetching
Next
From: Jacob Champion
Date:
Subject: Re: unclear OAuth error message