Skip to content

MLE-22657 Excluding beanutils from example project #1786

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

Merged
merged 1 commit into from
Jul 9, 2025

Conversation

rjrudin
Copy link
Contributor

@rjrudin rjrudin commented Jul 9, 2025

Turns out it's not even needed by the opencsv dependency.

Turns out it's not even needed by the opencsv dependency.
@Copilot Copilot AI review requested due to automatic review settings July 9, 2025 15:59
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the example project's Gradle build to remove an unused and vulnerable transitive dependency from the OpenCSV library.

  • Bump com.opencsv:opencsv from 5.11.1 to 5.11.2
  • Exclude the commons-beanutils module due to a security vulnerability
Comments suppressed due to low confidence (1)

examples/build.gradle:23

  • Consider referencing the specific CVE or advisory ID in the comment to document the exact security vulnerability for future maintainers.
		// Excluding this due to a security vulnerability, and the test for the example that uses this library

api ('com.opencsv:opencsv:5.11.2') {
// Excluding this due to a security vulnerability, and the test for the example that uses this library
// passes without this on the classpath.
exclude module: "commons-beanutils"
Copy link
Preview

Copilot AI Jul 9, 2025

Choose a reason for hiding this comment

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

For clarity and to avoid accidental exclusions, explicitly specify both group and module in the exclude clause, e.g., exclude group: 'commons-beanutils', module: 'commons-beanutils'.

Suggested change
exclude module: "commons-beanutils"
exclude group: "commons-beanutils", module: "commons-beanutils"

Copilot uses AI. Check for mistakes.

@rjrudin rjrudin merged commit 9bf534b into develop Jul 9, 2025
1 of 2 checks passed
@rjrudin rjrudin deleted the feature/22657-bean-utils branch July 9, 2025 17:42
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.

2 participants