Learn more about these different git repos.
Other Git URLs
3d1d5a8
@@ -104,7 +104,7 @@
context = contextlib.nullcontext()
if args.daemon:
- context = daemon.DaemonContext()
+ context = daemon.DaemonContext(umask=0o022)
with context:
logger_name = '{}.{}.pid-{}'.format(
Fixes: #1140
rebased onto 3d1d5a8
I plan to merge this right after first LGTM, and create micro release with this.
By default, the umask should be inherited from its parent process, so I would expect it to have some reasonable value. Interesting. This change can't hurt though, +1.
Ok, thanks .. I'll point the commit message to documentation about the insane defaults and merge. Thanks for the review!
Commit d849108 fixes this pull-request
Pull-Request has been merged by praiskup
Fixes: #1140