Skip to content

version bumps to handle CVE's - #29902

Open
thomas-temby wants to merge 2 commits into
Azure:mainfrom
thomas-temby:cve-version-bumps
Open

version bumps to handle CVE's#29902
thomas-temby wants to merge 2 commits into
Azure:mainfrom
thomas-temby:cve-version-bumps

Conversation

@thomas-temby

Copy link
Copy Markdown
Contributor

Description

Mandatory Checklist

  • SHOULD update ChangeLog.md file(s) appropriately
    • Update src/{{SERVICE}}/{{SERVICE}}/ChangeLog.md.
      • A snippet outlining the change(s) made in the PR should be written under the ## Upcoming Release header in the past tense.
    • Should not change ChangeLog.md if no new release is required, such as fixing test case only.
  • SHOULD regenerate markdown help files if there is cmdlet API change. Instruction
  • SHOULD have proper test coverage for changes in pull request.
  • SHOULD NOT adjust version of module manually in pull request

Copilot AI review requested due to automatic review settings July 27, 2026 07:36
@azure-pipelines

Copy link
Copy Markdown
Contributor
Azure Pipelines:
Successfully started running 1 pipeline(s).
There may be pipelines that require an authorized user to comment /azp run to run.

@thomas-temby

Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Contributor
Azure Pipelines:
Successfully started running 3 pipeline(s).

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 aims to address security CVEs by bumping dependency versions used by the repo’s tooling and build/test infrastructure.

Changes:

  • Updated fast-uri version metadata under the ajv entry in the MCP tool’s package-lock.json.
  • Bumped System.Security.Cryptography.Xml from 8.0.3 to 8.0.4 in Az.nonprod.props (used by net8.0 tool/test builds).

Reviewed changes

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

File Description
tools/Mcp/package-lock.json Adjusts transitive dependency version metadata for the MCP tool’s npm lockfile.
tools/Az.nonprod.props Updates a .NET package reference used by non-production/tooling builds.
Files not reviewed (1)
  • tools/Mcp/package-lock.json: Generated file

Comment thread tools/Mcp/package-lock.json
@a0x1ab

a0x1ab commented Jul 27, 2026

Copy link
Copy Markdown
Member

/azp run

@azure-pipelines

Copy link
Copy Markdown
Contributor
Azure Pipelines:
Successfully started running 3 pipeline(s).

Copilot AI review requested due to automatic review settings July 27, 2026 23:02
@a0x1ab

a0x1ab commented Jul 27, 2026

Copy link
Copy Markdown
Member

/azp run

@azure-pipelines

Copy link
Copy Markdown
Contributor
Azure Pipelines:
Successfully started running 3 pipeline(s).

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 1 out of 2 changed files in this pull request and generated 1 comment.

Files not reviewed (1)
  • tools/Mcp/package-lock.json: Generated file
Comments suppressed due to low confidence (1)

tools/Mcp/package-lock.json:40

  • These lockfile entries use an internal Azure DevOps feed URL and a sha1 integrity hash, while the rest of the lockfile uses registry.npmjs.org URLs with sha512 integrity. This can make installs fail for contributors/CI environments without access to that feed and weakens package integrity verification. Regenerate the lockfile using the standard registry (or ensure all entries consistently use the intended registry) so npm records a sha512 integrity for this package.

Comment on lines +26 to +28
"version": "1.19.14",
"resolved": "https://ms-feed-12.pkgs.visualstudio.com/1es-public/_packaging/npm-public/npm/registry/@hono/node-server/-/node-server-1.19.14.tgz",
"integrity": "sha1-4w+ES8d+POe+RCqsOx9zrYtY0YE=",
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