Skip to content
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

Allow other logging handlers #187

Closed
bpfoster opened this issue Dec 24, 2014 · 1 comment
Closed

Allow other logging handlers #187

bpfoster opened this issue Dec 24, 2014 · 1 comment

Comments

@bpfoster
Copy link
Contributor

@bpfoster bpfoster commented Dec 24, 2014

Currently com.twitter.finatra.Logging hijacks the logging configuration in a premain. This works well for quickly running out of the box, but less so when custom configuration is desired. For example, this makes bridging Java logging over SLF4j impossible.

@caniszczyk
Copy link
Contributor

@caniszczyk caniszczyk commented May 1, 2015

The new Finatra 2.0.0 M1 has logback which uses SLF4J so you should be OK now

@caniszczyk caniszczyk closed this May 1, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants