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.

Sign up for GitHub

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

Jump to bottom

Config directory mode #131

Closed
pixelrebel opened this issue Jan 13, 2023 · 1 comment · Fixed by #132
Closed

Config directory mode #131

pixelrebel opened this issue Jan 13, 2023 · 1 comment · Fixed by #132

Comments

@pixelrebel
Copy link

There is no reason to set the config directory to 700. When installing in a very common location like /etc/rclone.conf, the role hoses the mode of the /etc directory and breaks a lot of stuff. 755 is a perfectly reasonable mode for a config directory as long as the file is set to 600.
@stefangweichinger
Copy link
Owner

Good observation, although we install to /root/.config/rclone per default and not to /etc. But yes, if you change that to /etc, that breaks things. I will come up with a PR soon. Thanks.

stefangweichinger added a commit that referenced this issue Jan 15, 2023
@stefangweichinger
fix(fix:-fixes-#131.-correct-permissions-of-config-directory): fix th…
375cbc4 
…e permissions to not break perms if config directory is something common like /etc
@stefangweichinger stefangweichinger mentioned this issue Jan 15, 2023
Merged
stefangweichinger added a commit that referenced this issue Jan 15, 2023
@stefangweichinger
fix(fix:-fixes-#131.-correct-permissions-of-config-directory): fix th…
13be952 
…e permissions to not break perms if config directory is something common like /etc (#132)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

correct-permissions-of-config-directory stefangweichinger/ansible-rclone
2 participants
@stefangweichinger @pixelrebel