Bug 271668 - net/syncthing: Patch to set umask
Summary: net/syncthing: Patch to set umask
Status: New
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Dan Langille
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-05-27 10:36 UTC by Daniel O'Connor
Modified: 2023-05-27 14:11 UTC (History)
2 users (show)

See Also:
bugzilla: maintainer-feedback? (dvl)

Attachments
Patch to add umask setting (2.22 KB, patch)
2023-05-27 10:36 UTC, Daniel O'Connor 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Daniel O'Connor 2023-05-27 10:36:24 UTC 
Created attachment 242439 [details]
Patch to add umask setting

Hi,
We use syncthing at work for various files and it is useful to control the umask, unfortunately this doesn't seem possible via daemon (I tried that first - https://reviews.freebsd.org/D20969 - something resets it and I couldn't work out what).

I tried getting this patch upstreamed (https://github.com/syncthing/syncthing/pull/5857) but they were not interested (for reasonable reasons I suppose).
Comment 1 Mina Galić freebsd_triage 2023-05-27 11:06:07 UTC 
adding Ihor to cc, who is currently working on daemon bugs
Comment 2 Dan Langille freebsd_committer freebsd_triage 2023-05-27 11:26:36 UTC 
I agree with the syncthing Github issue: this is better handled by the startup scripts (if it can be) than within the code or with daemon.

However, I don't know how to achieve this within the rc.d scripts for syncthing.
Comment 3 Ihor Antonov 2023-05-27 14:11:44 UTC 
seems to be relatively straightforward. I will add it to my backlog