Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove hardcoded chmod from the code and use normal umask for permissions #279

Closed
subbink opened this issue Feb 20, 2023 · 1 comment · Fixed by #281
Closed

Remove hardcoded chmod from the code and use normal umask for permissions #279

subbink opened this issue Feb 20, 2023 · 1 comment · Fixed by #281

Comments

@subbink
Copy link

subbink commented Feb 20, 2023

At the moment all files written by dsc have the 0644 permissions. This makes it impossible to change the permissions based on the umask.
I would like to be able to use umask, so I don't have world readable files.

@jelu
Copy link
Member

jelu commented Mar 6, 2023

I will add new conf option to control output file access, changing it to use umask would break backwards compatibility.

jelu added a commit to jelu/dsc that referenced this issue Mar 6, 2023
- Fix DNS-OARC#279: Add new conf options to control output file access:
  - `output_user`: set output file user ownership
  - `output_group`: set output file group ownership
  - `output_mod`: set output file mode bits
@jelu jelu mentioned this issue Mar 6, 2023
jelu added a commit to jelu/dsc that referenced this issue Mar 6, 2023
- Fix DNS-OARC#279: Add new conf options to control output file access:
  - `output_user`: set output file user ownership
  - `output_group`: set output file group ownership
  - `output_mod`: set output file mode bits
jelu added a commit to jelu/dsc that referenced this issue Mar 6, 2023
- Fix DNS-OARC#279: Add new conf options to control output file access:
  - `output_user`: set output file user ownership
  - `output_group`: set output file group ownership
  - `output_mod`: set output file mode bits
@jelu jelu closed this as completed in #281 Mar 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants