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

Fix permissions on module files #3645

Merged
merged 2 commits into from
Feb 22, 2017

Conversation

tsg
Copy link
Contributor

@tsg tsg commented Feb 22, 2017

They contain configuration files, so they shouldn't be group writable,
which was the case before this patch.

Fixes #3644.

They contain configuration files, so they shouldn't be group writable,
which was the case before this patch.

Fixes elastic#3644.
@tsg tsg added needs_backport PR is waiting to be backported to other branches. review v5.3.0 labels Feb 22, 2017
@tsg
Copy link
Contributor Author

tsg commented Feb 22, 2017

jenkins, package it

@tsg
Copy link
Contributor Author

tsg commented Feb 22, 2017

Packaging tests are green: http://build-eu-00.elastic.co/job/beats-package-PR/

@monicasarbu monicasarbu merged commit c27250f into elastic:master Feb 22, 2017
tsg added a commit to tsg/beats that referenced this pull request Feb 22, 2017
* Fix permissions on module files

They contain configuration files, so they shouldn't be group writable,
which was the case before this patch.

Fixes elastic#3644.

(cherry picked from commit c27250f)
@tsg tsg removed the needs_backport PR is waiting to be backported to other branches. label Feb 22, 2017
urso pushed a commit that referenced this pull request Feb 22, 2017
* Fix permissions on module files

They contain configuration files, so they shouldn't be group writable,
which was the case before this patch.

Fixes #3644.

(cherry picked from commit c27250f)
tsg pushed a commit to tsg/beats that referenced this pull request Apr 3, 2017
The fix in elastic#3645 had a bug (chmod executed on the wrong folder). This fixes the fix and also adds permissions checks to the tests.
andrewkroh pushed a commit that referenced this pull request Apr 3, 2017
The fix in #3645 had a bug (chmod executed on the wrong folder). This fixes the fix and also adds permissions checks to the tests.
tsg added a commit to tsg/beats that referenced this pull request Apr 5, 2017
The fix in elastic#3645 had a bug (chmod executed on the wrong folder). This fixes the fix and also adds permissions checks to the tests.
(cherry picked from commit 37ae2fc)
ruflin pushed a commit that referenced this pull request Apr 6, 2017
The fix in #3645 had a bug (chmod executed on the wrong folder). This fixes the fix and also adds permissions checks to the tests.
(cherry picked from commit 37ae2fc)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants