Skip to content
This repository has been archived by the owner on May 10, 2022. It is now read-only.

remove log4j from dependencies. #68

Open
oujinliang opened this issue Nov 14, 2019 · 2 comments
Open

remove log4j from dependencies. #68

oujinliang opened this issue Nov 14, 2019 · 2 comments

Comments

@oujinliang
Copy link

I found this java client depends on log4j:

    <dependency>
      <groupId>org.slf4j</groupId>
      <artifactId>slf4j-log4j12</artifactId>
      <version>1.7.2</version>
    </dependency>

it's not good to bind a slf4j implementation like log4j in the library, many applications use other slf4j bindings like logback or log4j2.

@foreverneverer
Copy link
Contributor

Thanks,we will remove it if it's reasonable and feasible

@foreverneverer
Copy link
Contributor

in #70, we are changing to log4j2. welcome to review it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants