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

stat_file when keep is set, instead of mirroring all file permissions #40609

Merged
merged 1 commit into from Apr 11, 2017

Conversation

gtmanfred
Copy link
Contributor

What does this PR do?

This changes the behavior of the fileserver which used to mirror permissions from the salt master to the minion cache when caching files.

What issues does this PR fix or reference?

Fixes #40075

Previous Behavior

Before this change, the file permissions from the fileserver were mirrored to the minion file cache.

New Behavior

Now the cache is set to 600, and we lookup the file permissions at the time of placing the file down on the filesystem.

Tests written?

No

remove chmod of files in the fileserver
add stat_file to cp module
use cp.stat_file to get file mode in file module
add a note about keep not working with ssh
@cachedout cachedout merged commit 8492cef into saltstack:2016.11 Apr 11, 2017
@msmeissn
Copy link

assigned CVE-2017-8109.

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 this pull request may close these issues.

None yet

4 participants