Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Resolve probable false positive with code scanning re: Druid driver #42446

Closed
1 task
crisptrutski opened this issue May 9, 2024 · 2 comments
Closed
1 task
Labels
.Task Not a part of any Epic, used by the Task Issue Template .Team/BackendComponents also known as BEC

Comments

@crisptrutski
Copy link
Contributor

This looks like a false positive:

  1. It refers to a pom file that doesn't exist in the repo.
  2. It refers to a different version of the vertica driver to the one we package.
  3. Printing out the dependency tree1 with the drivers on the class path doesn't show the old version of the affected package.

Tracking issue for:

Footnotes

  1. clj -A:dev:ee:ee-dev:drivers:drivers-dev:build -Stree

@crisptrutski
Copy link
Contributor Author

Created this to avoid wasting other people's time on investigating. Hopefully the scanner auto-resolves this shortly.

@darksciencebase darksciencebase added .Team/BackendComponents also known as BEC .Task Not a part of any Epic, used by the Task Issue Template labels May 10, 2024
@piranha
Copy link
Contributor

piranha commented May 16, 2024

Pom files do not exist since they are created by a .github/script/write-poms.sh specifically to track deps with Snyk. :) But pom.xml in question refers org.clojure/clojure 1.11.2 explicitly, so it doesn't really make any sense to me - I'm going to dismiss the alert.

@piranha piranha closed this as completed May 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
.Task Not a part of any Epic, used by the Task Issue Template .Team/BackendComponents also known as BEC
Projects
None yet
Development

No branches or pull requests

3 participants