-
Notifications
You must be signed in to change notification settings - Fork 31
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
Template should accept file permissions #31
Comments
What were you envisioning for this? Something like the following?
Also: why not just accomplish this with something like |
Because if we're rendering sensitive information to disk it needs to write a more secure default. Right now it's 0755ing everything, which is not ideal. It could be 0600 by default and then expect the user to specify mode separately, you're right. In your code sketch, we'd just have mode. Maybe this needs to be a separate issue. |
@BrianHicks That makes sense, I guess you never want secrets on disk with loose permissions (even for a second). I prefer the option of documenting the default |
Closing in favor of those referenced above. |
These should be templated as well, but user and group should be separate resources.
The text was updated successfully, but these errors were encountered: