Skip to content

Commit

Permalink
fix snakeyaml vulnerability issue by disabling detekt (#237)
Browse files Browse the repository at this point in the history
Signed-off-by: Subhobrata Dey <sbcd90@gmail.com>
  • Loading branch information
sbcd90 committed Sep 7, 2022
1 parent dea1aed commit e24d151
Showing 1 changed file with 7 additions and 4 deletions.
11 changes: 7 additions & 4 deletions build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ buildscript {
classpath "${opensearch_group}.gradle:build-tools:${opensearch_version}"
classpath "org.jetbrains.kotlin:kotlin-gradle-plugin:${kotlin_version}"
classpath "org.jetbrains.kotlin:kotlin-allopen:${kotlin_version}"
classpath "io.gitlab.arturbosch.detekt:detekt-gradle-plugin:1.20.0-RC1"
// classpath "io.gitlab.arturbosch.detekt:detekt-gradle-plugin:1.20.0-RC1"
}
}

Expand Down Expand Up @@ -59,14 +59,15 @@ apply plugin: 'jacoco'
apply plugin: 'signing'
apply plugin: 'maven-publish'
apply plugin: 'com.github.johnrengelman.shadow'
apply plugin: 'io.gitlab.arturbosch.detekt'
// apply plugin: 'io.gitlab.arturbosch.detekt'
apply plugin: 'org.jetbrains.kotlin.jvm'
apply plugin: 'org.jetbrains.kotlin.plugin.allopen'
apply plugin: 'opensearch.repositories'
apply from: 'build-tools/opensearchplugin-coverage.gradle'

configurations {
ktlint
all*.exclude group: 'org.yaml', module: 'snakeyaml'
}

dependencies {
Expand Down Expand Up @@ -103,10 +104,12 @@ spotless {
eclipse().configFile rootProject.file('.eclipseformat.xml')
}
}
detekt {

// TODO: enable detekt only when snakeyaml vulnerability is fixed
/*detekt {
config = files("detekt.yml")
buildUponDefaultConfig = true
}
}*/

task ktlint(type: JavaExec, group: "verification") {
description = "Check Kotlin code style."
Expand Down

0 comments on commit e24d151

Please sign in to comment.