Skip to content

fix: security alerts and align pom dependency properties - #28

Merged
guanshengliang merged 5 commits into
mainfrom
fix/6604269742
Apr 23, 2026
Merged

fix: security alerts and align pom dependency properties#28
guanshengliang merged 5 commits into
mainfrom
fix/6604269742

Conversation

@sheyanjie-qq

@sheyanjie-qq sheyanjie-qq commented Apr 22, 2026

Copy link
Copy Markdown
Contributor

Description

fix security alerts and align pom dependency properties

Issue(s)

Checklist

Please check the items in the checklist if applicable.

  • Is the user manual updated?
  • Are the test cases passed and automated?
  • Is there no significant decrease in test coverage?

sheyanjie-taos and others added 3 commits April 22, 2026 14:22
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings April 22, 2026 06:52
@gemini-code-assist

Copy link
Copy Markdown

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates dependency versions/properties and GitHub Actions permissions to address security alerts and standardize dependency management across build and deploy POMs.

Changes:

  • Bump Jackson version and centralize Log4j/LZ4 versions via Maven properties in both pom.xml and deploy-pom.xml.
  • Replace/standardize LZ4 usage by excluding org.lz4:lz4-java from Flink deps and adding at.yawk.lz4:lz4-java.
  • Add explicit permissions blocks to GitHub Actions workflows.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
pom.xml Updates dependency properties (Jackson/Log4j/LZ4) and adjusts LZ4 dependency/exclusions to mitigate CVEs.
deploy-pom.xml Mirrors pom.xml dependency property alignment and LZ4 dependency/exclusions.
.github/workflows/release.yml Adds workflow token permissions and fixes YAML indentation for the dependency submission step.
.github/workflows/build.yml Adds explicit read-only workflow token permissions.

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

Comment thread .github/workflows/release.yml Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings April 22, 2026 06:56

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.


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

Comment thread .github/workflows/release.yml
@codecov-commenter

codecov-commenter commented Apr 22, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.35%. Comparing base (ec9846c) to head (e1eb937).

Additional details and impacted files
@@            Coverage Diff            @@
##               main      #28   +/-   ##
=========================================
  Coverage     80.35%   80.35%           
  Complexity      468      468           
=========================================
  Files            56       56           
  Lines          2016     2016           
  Branches        233      233           
=========================================
  Hits           1620     1620           
  Misses          285      285           
  Partials        111      111           
Flag Coverage Δ
unittests 80.35% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

@zyyang90 zyyang90 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@guanshengliang
guanshengliang merged commit 06af6f7 into main Apr 23, 2026
1 check passed
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.

6 participants