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

(PUP-10627) Add default file permissions 640 for last_run_summary.yaml #8433

Merged
merged 1 commit into from
Nov 4, 2020

Conversation

luchihoratiu
Copy link
Contributor

@luchihoratiu luchihoratiu commented Nov 4, 2020

Due to security concerns, this commit downgrades file permissions for last_run_summary.yaml from 644 to 640. This allows access only to file owner and its group (by default).

See #8277 and ticket comments for more information.

Due to security concerns, this commit downgrades file permissions for
`last_run_summary.yaml` from `644` to `640`. This allows only file owner
and its group access to it by default.
@luchihoratiu luchihoratiu requested review from a team November 4, 2020 10:28
@luchihoratiu
Copy link
Contributor Author

Jenkins please test this on debian9-32a,debian10-64a,fedora30-64a,fedora31-64a,windows2016-64a,windows2019-64a,centos6-64a,centos6-32a,redhat7-64a,redhat7-AARCH64a,redhat8-64a,redhatfips7-64a,windows10ent-32a

@GabrielNagy GabrielNagy merged commit f664d6a into puppetlabs:main Nov 4, 2020
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.

2 participants