Skip to content

Removes vulnerable package#160

Merged
JoonLeeNIH merged 2 commits into3.3.0from
vuln-fix
Mar 13, 2026
Merged

Removes vulnerable package#160
JoonLeeNIH merged 2 commits into3.3.0from
vuln-fix

Conversation

@JoonLeeNIH
Copy link
Contributor

Overview

Vulnerability fix

Change Details (Specifics)

Fixes jackson dependency vulnerability

Related Ticket(s)

None

Copy link

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

Updates Maven dependency management to address a Jackson-related vulnerability by centrally managing Jackson versions via a BOM import.

Changes:

  • Import com.fasterxml.jackson:jackson-bom in dependencyManagement to align Jackson artifact versions.
  • Remove the explicit version from jackson-dataformat-yaml so it is managed by the BOM.
Comments suppressed due to low confidence (1)

pom.xml:46

  • The indentation/formatting in this dependencyManagement block is inconsistent: netty-codec’s <groupId>/<artifactId>/<version> lines aren’t aligned with the surrounding XML. Please reformat this section for readability and to avoid future noisy diffs.
            </dependency>
            <dependency>
            <groupId>io.netty</groupId>
            <artifactId>netty-codec</artifactId>
            <version>4.1.125.Final</version>

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@JoonLeeNIH JoonLeeNIH changed the title Fixes jackson vulnerability by copying C3DC Removes vulnerable package Mar 13, 2026
Copy link
Contributor

@huuaho huuaho left a comment

Choose a reason for hiding this comment

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

LGTM

@JoonLeeNIH JoonLeeNIH merged commit ceabb0b into 3.3.0 Mar 13, 2026
2 of 3 checks passed
@JoonLeeNIH JoonLeeNIH deleted the vuln-fix branch March 13, 2026 15:37
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