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

Revert node-id file validations #4433

Merged
merged 2 commits into from Dec 20, 2017

Conversation

Projects
None yet
2 participants
@bernd
Member

bernd commented Dec 20, 2017

We revert this for 2.4 because we ran into issues and edge cases twice. So for the 2.4 release we don't want to risk any issues. We keep the changes in 3.0 and create another fix.

bernd added some commits Dec 20, 2017

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

This reverts commit 204a624.

@bernd bernd added this to the 2.4.0 milestone Dec 20, 2017

@bernd bernd requested a review from kroepke Dec 20, 2017

@kroepke kroepke merged commit 018d397 into 2.4 Dec 20, 2017

1 of 5 checks passed

ci-web-linter Jenkins build graylog-pr-linter-check 2139 has failed
Details
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
continuous-integration/travis-ci/push The Travis CI build is in progress
Details
graylog-project/pr Jenkins build graylog-project-pr-snapshot 842 is running
Details
license/cla Contributor License Agreement is signed.
Details

@kroepke kroepke deleted the revert-4417-and-4428 branch Dec 20, 2017

@wafflebot wafflebot bot removed the ready-for-review label Dec 20, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment