Skip to content
This repository has been archived by the owner on Apr 1, 2024. It is now read-only.

ISSUE-19484: [Bug] pulsar-client-all 2.11.0 transitively bringing in log4j2 implementation #5509

Open
1 of 2 tasks
sijie opened this issue Feb 10, 2023 · 0 comments
Open
1 of 2 tasks

Comments

@sijie
Copy link
Member

sijie commented Feb 10, 2023

Original Issue: apache#19484


Search before asking

  • I searched in the issues and found nothing similar.

Version

  • OS: <not applicable>
  • Pulsar: 2.11.0

Details

Problem

The pulsar-client-all:2.11.0 dependency transitively includes Log4j2 (see here) rather than just the SLF4J api and logging bridge.
Whereas the pulsar-client-all:2.10.3 does NOT bring a logging impl in (see here).

Symptoms

If the consuming app is already using a logger impl then a conflict can occur at startup (when using SLF4J) telling user to pick one or the other.

If the consuming application is not already using a logger impl then pulsar-client-all effectively chooses LOG4J2 for the app.

What did you expect to see?

Adding the dependency org.apache.pulsar:pulsar-client-all:2.11.0 should NOT transitively bring in the log4j2 implementation.

What did you see instead?

Adding the dependency org.apache.pulsar:pulsar-client-all:2.11.0 transitively DID bring in the log4j2 implementation.

Anything else?

No response

Are you willing to submit a PR?

  • I'm willing to submit a PR!
@sijie sijie added the type/bug label Feb 10, 2023
@sijie sijie added the Stale label Mar 13, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant