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 node id file permission checks #4428

Merged
merged 1 commit into from Dec 18, 2017
Merged

fix node id file permission checks #4428

merged 1 commit into from Dec 18, 2017

Conversation

kroepke
Copy link
Member

@kroepke kroepke commented Dec 18, 2017

server start would fail if the node id was non-empty, but not writable.
this is a valid scenario where the node id is frozen by making the file read-only

added tests for various scenarios

fixes #4427

server start would fail if the node id was non-empty, but not writable.
this is a valid scenario where the node id is frozen by making the file read-only

added tests for various scenarios

fixes #4427
@kroepke kroepke added this to the 2.4.0 milestone Dec 18, 2017
@kroepke kroepke requested a review from bernd December 18, 2017 10:25
@ghost ghost assigned kroepke Dec 18, 2017
@bernd bernd assigned bernd and unassigned kroepke Dec 18, 2017
Copy link
Member

@bernd bernd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@bernd bernd merged commit 3de61c0 into master Dec 18, 2017
@ghost ghost removed the ready-for-review label Dec 18, 2017
@bernd bernd deleted the issue-4427 branch December 18, 2017 12:00
bernd pushed a commit that referenced this pull request Dec 18, 2017
server start would fail if the node id was non-empty, but not writable.
this is a valid scenario where the node id is frozen by making the file read-only

added tests for various scenarios

fixes #4427

(cherry picked from commit 3de61c0)
bernd added a commit that referenced this pull request Dec 20, 2017
kroepke pushed a commit that referenced this pull request Dec 20, 2017
* Revert "fix node id file permission checks (#4428)"

This reverts commit b0b7b9b.

* Revert "check node id file permissions as part of parameter validation, instead of failing during injection (#4417)"

This reverts commit 204a624.
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.

Upgrade to graylog-server-2.4.0-6.beta.4.noarch generates "Invalid configuration" due to file perm.
2 participants