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

Update reader base permissions #1902

Merged
merged 1 commit into from Mar 4, 2016

Conversation

Projects
None yet
2 participants
@kroepke
Member

kroepke commented Mar 4, 2016

Reader base permissions now include clusterconfigentry:read.

Without the fix:

  • Create stream
  • Create role granting access to the stream
  • Assign new role to reader user
  • As reader user try to search in stream
  • User is logged out and sees error notification

After fix:

  • Restart server
  • With the same reader user try to access stream
  • User can search in stream

fix #1887

Reader base permissions must now include clusterconfigentry:read to b…
…e able to retrieve the settings for the relative time dropdown.

fix #1887

@kroepke kroepke added this to the 2.0.0 milestone Mar 4, 2016

@bernd bernd self-assigned this Mar 4, 2016

@bernd

This comment has been minimized.

Member

bernd commented Mar 4, 2016

LGTM 👍

bernd added a commit that referenced this pull request Mar 4, 2016

Merge pull request #1902 from Graylog2/issue-1887
Update reader base permissions

@bernd bernd merged commit 6087425 into master Mar 4, 2016

2 of 4 checks passed

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
ci-server-integration Jenkins build graylog2-server-integration-pr 716 has succeeded
Details
ci-web-linter Jenkins build graylog-pr-linter-check 207 has succeeded
Details

@bernd bernd deleted the issue-1887 branch Mar 4, 2016

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