Skip to content

Conversation

@snyk-bot
Copy link
Contributor

Description

This PR fixes one or more vulnerable packages in the maven dependencies of this project.
See the Snyk test report for more details.

Snyk Project: Azure/azure-cosmosdb-java:benchmark/pom.xml

Snyk Organization: Azure

Changes included in this PR

  • Changes to the following files to upgrade the vulnerable dependencies to a fixed version:
    • benchmark/pom.xml

Vulnerabilities that will be fixed

With an upgrade:

You can read more about Snyk's upgrade and patch logic in Snyk's documentation.

Check the changes in this PR to ensure they won't cause issues with your project.

Stay secure,
The Snyk team

Note: You are seeing this because you or someone else with access to this repository has authorised Snyk to open Fix PRs. To review the settings for this Snyk project please go to the project settings page.

@moderakh
Copy link
Contributor

moderakh commented Nov 6, 2019

/azp run

@moderakh moderakh self-requested a review November 6, 2019 06:17
Copy link
Contributor

@moderakh moderakh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

there is an issue with version upgrade in the compilation log:

[ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.6.0:compile (default-compile) on project azure-cosmosdb-benchmark: Compilation failure
[ERROR] /home/vsts/work/1/s/benchmark/src/main/java/com/microsoft/azure/cosmosdb/benchmark/Main.java:[29,28] cannot access com.beust.jcommander.JCommander
[ERROR]   bad class file: /home/vsts/.m2/repository/com/beust/jcommander/1.75/jcommander-1.75.jar(com/beust/jcommander/JCommander.class)
[ERROR]     class file has wrong version 55.0, should be 52.0
[ERROR]     Please remove or make sure it appears in the correct subdirectory of the classpath.
[ERROR] 
[ERROR] -> [Help 1]

@moderakh
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants