Skip to content

Apply code review feedback: scope CVE suppressions and fix indentation#316

Merged
DovOps merged 1 commit intofeature/nov-25-dependency-fixesfrom
copilot/sub-pr-315
Nov 20, 2025
Merged

Apply code review feedback: scope CVE suppressions and fix indentation#316
DovOps merged 1 commit intofeature/nov-25-dependency-fixesfrom
copilot/sub-pr-315

Conversation

Copy link

Copilot AI commented Nov 20, 2025

Addresses code review feedback from PR #315 on Java dependency updates.

Changes

  • CVE suppressions: Added filePath regex to scope CVE-2022-41940 and CVE-2020-36048 suppressions to engine.io-client library only, preventing global application
  • Code formatting: Replaced tab with spaces in position-service/build.gradle for consistent indentation

Note: OkHttp 5.3.0 upgrade feedback was resolved as compatible per local testing.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

@netlify
Copy link

netlify bot commented Nov 20, 2025

Deploy Preview for lucky-concha-f3599f canceled.

Name Link
🔨 Latest commit 396ab91
🔍 Latest deploy log https://app.netlify.com/projects/lucky-concha-f3599f/deploys/691ed6aa4e177800087be371

Copilot AI changed the title [WIP] Update vulnerable java dependencies and reset suppressions Apply code review feedback: scope CVE suppressions and fix indentation Nov 20, 2025
Copilot AI requested a review from DovOps November 20, 2025 08:59
@DovOps DovOps marked this pull request as ready for review November 20, 2025 09:02
@DovOps DovOps merged commit 9714622 into feature/nov-25-dependency-fixes Nov 20, 2025
6 checks 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.

2 participants