opensearch-project / observability

Visualize and explore your logs, traces and metrics data in OpenSearch Dashboards
https://opensearch.org/docs/latest/observability-plugin/index/
Apache License 2.0
53 stars 95 forks source link

fix cve issues related to logback #1791

Open YANG-DB opened 5 months ago

YANG-DB commented 5 months ago

Description

fix cve issues related to logback

Issues Resolved

1780

Check List

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license. For more information on following Developer Certificate of Origin and signing off your commits, please check here.

joshuali925 commented 5 months ago

CI failed

Execution failed for task ':ktlint'.
> Could not resolve all dependencies for configuration ':ktlint'.
   > Conflict found for the following module:
       - org.slf4j:slf4j-api between versions 2.0.0 and 1.7.32
sejli commented 5 months ago

Full scan here

Dependency resolution failed because of conflict on the following module:
   - org.slf4j:slf4j-api between versions 2.0.0 and 1.7.32

org.slf4j:slf4j-api:2.0.0
  Variant runtime:
    | Attribute Name             | Provided     | Requested |
    |----------------------------|--------------|-----------|
    | org.gradle.category        | library      |           |
    | org.gradle.libraryelements | jar          |           |
    | org.gradle.status          | release      |           |
    | org.gradle.usage           | java-runtime |           |
   Selection reasons:
      - By conflict resolution: between versions 2.0.0 and 1.7.32

org.slf4j:slf4j-api:2.0.0
\--- ch.qos.logback:logback-classic:1.3.0
     \--- com.pinterest:ktlint:0.47.1
          \--- ktlint

org.slf4j:slf4j-api:1.7.32 -> 2.0.0
\--- io.github.microutils:kotlin-logging-jvm:2.1.23
     +--- com.pinterest.ktlint:ktlint-ruleset-standard:0.47.1
     |    \--- com.pinterest:ktlint:0.47.1
     |         \--- ktlint
     \--- com.pinterest.ktlint:ktlint-core:0.47.1
          +--- com.pinterest:ktlint:0.47.1 (*)
          +--- com.pinterest.ktlint:ktlint-reporter-baseline:0.47.1
          |    \--- com.pinterest:ktlint:0.47.1 (*)
          +--- com.pinterest.ktlint:ktlint-reporter-checkstyle:0.47.1
          |    \--- com.pinterest:ktlint:0.47.1 (*)
          +--- com.pinterest.ktlint:ktlint-reporter-json:0.47.1
          |    \--- com.pinterest:ktlint:0.47.1 (*)
          +--- com.pinterest.ktlint:ktlint-reporter-format:0.47.1
          |    \--- com.pinterest:ktlint:0.47.1 (*)
          +--- com.pinterest.ktlint:ktlint-reporter-html:0.47.1
          |    \--- com.pinterest:ktlint:0.47.1 (*)
          +--- com.pinterest.ktlint:ktlint-reporter-plain:0.47.1
          |    \--- com.pinterest:ktlint:0.47.1 (*)
          +--- com.pinterest.ktlint:ktlint-reporter-sarif:0.47.1
          |    \--- com.pinterest:ktlint:0.47.1 (*)
          +--- com.pinterest.ktlint:ktlint-ruleset-experimental:0.47.1
          |    \--- com.pinterest:ktlint:0.47.1 (*)
          +--- com.pinterest.ktlint:ktlint-ruleset-standard:0.47.1 (*)
          \--- com.pinterest.ktlint:ktlint-ruleset-test:0.47.1
               \--- com.pinterest:ktlint:0.47.1 (*)

(*) - Indicates repeated occurrences of a transitive dependency subtree. Gradle expands transitive dependency subtrees only once per project; repeat occurrences only display the root of the subtree, followed by this annotation.
ps48 commented 5 months ago

Added backport 2.12 label as the branch is cut for build team to proceed with RC today. @YANG-DB Can you please make sure after gradle issue is fixed, we backport this to 2.12?