/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

Changing subsystem on Logging page changes all subsystems #1904

Closed
bernd opened this Issue Mar 4, 2016 · 0 comments

Comments

Assignees

@joschi joschi

Labels
Projects
None yet
Milestone
  2.1.0
4 participants
@bernd @kroepke @joschi @dennisoelkers
@bernd
Member

bernd commented Mar 4, 2016

Problem description

Changing any Subsystem on the Logging page (except "Subsystem: Graylog") changes the level for all subsystems.

Steps to reproduce the problem

  1. Go to System/Logging
  2. Select "debug" for "Subsystem: Indexer"
  3. Log level for "Sockets", "Indexer" and "Authentication" changes to "debug"
  4. Check API request "/system/loggers/subsystems", same issue (so it's not a UI problem)

Analysis

It looks like the LoggersResource#setLoggerLevel() method changes the root logger instead of the requested class because there is no explicit config for that. The graylog subsystem does not change because our default log4j2.xml has an explicit configuration for that.

I guess this is a behaviour change in log4j2.

Environment

  • Graylog Version: 2.0.0-alpha.5
  • Elasticsearch Version: 2.2.0
  • MongoDB Version: 2.4.14
  • Operating System: Ubuntu 14.04

@bernd bernd added bug S3 P3 labels Mar 4, 2016

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

@kroepke kroepke modified the milestones: 2.0.0, 2.1.0 May 2, 2016

joschi added a commit that referenced this issue May 24, 2016

@joschi
Fix changing log level via REST API 
Fixes #1904
Loading status checks…
c253834

@joschi joschi self-assigned this May 24, 2016

joschi added a commit that referenced this issue May 24, 2016

@joschi
Fix changing log level via REST API 
Fixes #1904
Loading status checks…
f4ef053

joschi added a commit that referenced this issue May 24, 2016

@joschi
Fix changing log level via REST API 
Fixes #1904
Loading status checks…
2046306

joschi added a commit that referenced this issue May 24, 2016

@joschi
Fix changing log level via REST API 
Fixes #1904
Loading status checks…
52f9e39

@joschi joschi referenced this issue May 24, 2016

Merged

Fix changing log level via REST API #2277

@dennisoelkers dennisoelkers added the in progress label May 24, 2016

dennisoelkers added a commit that referenced this issue May 24, 2016

@joschi @dennisoelkers
Fix changing log level via REST API (#2277) 
Fixes #1904
Loading status checks…
dbcd445

@dennisoelkers dennisoelkers closed this in 7389454 May 24, 2016

@kroepke kroepke added the triaged label Sep 21, 2016

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