Skip to content

Commit

Permalink
[Fix](log)To avoid SLF4J multiple binding issues and unnecessary depe…
Browse files Browse the repository at this point in the history
…ndency propagation.

SLF4J requires only one binding implementatio to be present at runtime.
By setting the scope of other bindings, such as slf4j-reload4j, to test, we ensure that only the explicitly specified binding, like log4j-slf4j-impl, is used in the production environment, preventing runtime conflicts.
  • Loading branch information
CalvinKirs committed Nov 20, 2024
1 parent 34383ad commit a372515
Showing 1 changed file with 9 additions and 0 deletions.
9 changes: 9 additions & 0 deletions fe/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -857,6 +857,15 @@ under the License.
<artifactId>json-simple</artifactId>
<version>${json-simple.version}</version>
</dependency>
<!--SLF4J requires only one binding implementation to be present at runtime.
By setting the scope of other bindings, such as slf4j-reload4j, to test,
we ensure that only the explicitly specified binding, like log4j-slf4j-impl,
is used in the production environment, preventing runtime conflicts.-->
<dependency>
<groupId>org.slf4j</groupId>
<artifactId>slf4j-reload4j</artifactId>
<scope>test</scope>
</dependency>
<!-- https://mvnrepository.com/artifact/org.junit.jupiter/junit-jupiter-engine -->
<dependency>
<groupId>org.junit.jupiter</groupId>
Expand Down

0 comments on commit a372515

Please sign in to comment.