/finatra

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

LogLevel doesn't seem to work #83

Closed
hgavert opened this Issue Jan 3, 2014 · 1 comment

Comments

Assignees
No one assigned
Labels
None yet
Milestone
  1.5.1
2 participants
@hgavert @capotej
@hgavert

hgavert commented Jan 3, 2014

Hi!

I've tried to set the logging level to DEBUG and WARNING or ERROR by

  1. using the flag: -com.twitter.finatra.config.logLevel='ERROR' or
  2. setting System.setProperty("com.twitter.finatra.config.logLevel", "DEBUG") before registering the controllers

However, I always seem to have info messages coming through and never debug. If I call log.getLevel, I always get null.

Maybe I'm doing something wrong and I'm stuck. BTW, the documentation about the flags is wrong: it's not -usage but -help seems to work :-)
@capotej

This comment has been minimized.

Sign in to view
Contributor

capotej commented Jan 3, 2014

Thanks for the bug report, there may have been a regression around logging in a recent release, going to harden this up here: #84

I'll fix the usage typo also 😄

@capotej capotej closed this in 635b4dc Jan 12, 2014

armandocanals added a commit to armandocanals/finatra that referenced this issue Jan 14, 2014

@capotej @armandocanals
Tests around logging and fix for setting logLevel, fixes twitter#83 and 
twitter#84
fa1a3ce

cacoco pushed a commit that referenced this issue May 13, 2015

@capotej
Tests around logging and fix for setting logLevel, fixes #83 and #84
74a7879

cacoco pushed a commit that referenced this issue May 13, 2015

@capotej @armandocanals
Tests around logging and fix for setting logLevel, fixes #83 and #84
47d6461
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment