/graylog2-server

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

Check for user permisson for recent internal log messages #4859

Closed
1Jesper1 opened this Issue Jun 22, 2018 · 0 comments

Comments

Assignees

@bernd bernd

Labels
Projects
None yet
Milestone
  3.0.0
3 participants
@1Jesper1 @dennisoelkers @bernd
@1Jesper1

1Jesper1 commented Jun 22, 2018

graylog2-server/graylog2-server/src/main/java/org/graylog2/rest/resources/system/logs/LoggersResource.java

Line 216 in 365d26f

public LogMessagesSummary messages(@ApiParam(name = "limit", value = "How many log messages should be returned", defaultValue = "500", allowableValues = "range[0, infinity]")

@dennisoelkers dennisoelkers added bug security triaged labels Jun 25, 2018

@dennisoelkers dennisoelkers added this to the 3.0.0 milestone Jun 25, 2018

@bernd bernd self-assigned this Jun 28, 2018

bernd added a commit that referenced this issue Jun 28, 2018

@bernd
Check permissions for recent log messages API endpoint 
Fixes #4859
Loading status checks…
dcb0061

@bernd bernd referenced this issue Jun 28, 2018

Merged

Add missing permission checks #4873

@dennisoelkers dennisoelkers closed this in #4873 Jun 28, 2018

dennisoelkers added a commit that referenced this issue Jun 28, 2018

@bernd @dennisoelkers
Add missing permission checks (#4873) 
* Add missing LOOKUP_TABLES_READ permission check

Fixes #4861

* Check permissions for recent log messages API endpoint

Fixes #4859
Loading status checks…
5492f63

dennisoelkers added a commit that referenced this issue Jun 28, 2018

@bernd @dennisoelkers
Add missing permission checks (#4873) 
* Add missing LOOKUP_TABLES_READ permission check

Fixes #4861

* Check permissions for recent log messages API endpoint

Fixes #4859
Verified
This commit was signed with a verified signature.
@dennisoelkers dennisoelkers Dennis Oelkers
GPG key ID: 440E58BD30A068C8 Learn about signing commits
Loading status checks…
9613bbf

@dennisoelkers dennisoelkers referenced this issue Jun 28, 2018

Merged

Add missing permission checks #4878

bernd added a commit that referenced this issue Jun 28, 2018

@dennisoelkers @bernd
Add missing permission checks (#4873) (#4878) 
* Add missing LOOKUP_TABLES_READ permission check

Fixes #4861

* Check permissions for recent log messages API endpoint

Fixes #4859
Loading status checks…
4f3eb82
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment