Skip to content

Commit

Permalink
Remove log4j from runtime classpath as it is only needed to generate …
Browse files Browse the repository at this point in the history
…a recipe from a refaster template

This does not yet work as expected in the test
  • Loading branch information
pstreef committed Dec 20, 2024
1 parent 8dc20c7 commit 3fec5a5
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion build.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,10 @@ recipeDependencies {
}

dependencies {
compileOnly("log4j:log4j:1.+") {
because("log4j 1 has critical vulnerabilities but we need the type for the refaster recipe during compilation")
}

compileOnly("org.projectlombok:lombok:latest.release")
annotationProcessor("org.projectlombok:lombok:latest.release")

Expand All @@ -31,7 +35,6 @@ dependencies {
implementation("org.openrewrite.recipe:rewrite-static-analysis:${rewriteVersion}")
runtimeOnly("org.openrewrite:rewrite-java-17")

implementation("log4j:log4j:1.+")
implementation("org.apache.logging.log4j:log4j-core:2.+")
implementation("org.slf4j:slf4j-api:2.+")

Expand Down

0 comments on commit 3fec5a5

Please sign in to comment.