Skip to content

Address dependency vulnerabilities: upgrade packages and fix related tests/compatibility#2647

Open
jainakanksha-msft wants to merge 17 commits into
akkuNodejsUpgradefrom
akkuSFI
Open

Address dependency vulnerabilities: upgrade packages and fix related tests/compatibility#2647
jainakanksha-msft wants to merge 17 commits into
akkuNodejsUpgradefrom
akkuSFI

Conversation

@jainakanksha-msft

@jainakanksha-msft jainakanksha-msft commented May 21, 2026

Copy link
Copy Markdown
Collaborator

Description

Introduced several important updates and improvements across the codebase, focusing on dependency upgrades, improved binary data handling, and test consistency. The changes enhance compatibility with newer library versions, standardize the use of Uint8Array for binary data, and streamline test logic.

Dependency and Compatibility Updates:

  • Upgraded multiple dependencies in package.json, including @azure/ms-rest-js, axios, tedious, @azure/storage-blob, and @azure/storage-queue to their latest versions for improved security and compatibility. Also added @azure/identity and removed the deprecated azure-storage package.

Binary Data Handling Improvements:

  • Standardized the use of Uint8Array instead of Buffer throughout the codebase for handling binary data, including MD5 hashes, content MD5, and internal buffer conversions. This affects blob metadata stores, response handlers, and utility functions.

Test Suite Consistency and Refactoring:

  • Updated tests to consistently use Uint8Array for binary data, removed unnecessary environment checks, and simplified credential retrieval logic in blob batch API tests. Also fixed a property name casing issue in an authentication error assertion.

Swagger and Documentation Updates:

  • Removed the default value for the BlobSequenceNumber parameter in the Swagger definition and updated the markdown documentation to reflect this change.

Minor Refactoring and Code Quality:

  • Improved code style and type safety in several places, such as updating method signatures to return this for stream end methods, fixing minor formatting, and refining type annotations.

Add Change Log

  • Updated the README.md
  • Updated the ChangeLog.md

Copilot AI review requested due to automatic review settings May 21, 2026 10:11

This comment was marked as outdated.

Copilot AI review requested due to automatic review settings May 27, 2026 08:04
Comment thread tests/table/utils/table.entity.test.utils.ts Dismissed
Comment thread tests/table/utils/table.entity.test.utils.ts Dismissed

This comment was marked as outdated.

Copilot AI review requested due to automatic review settings May 28, 2026 08:03

This comment was marked as outdated.

Copilot AI review requested due to automatic review settings May 28, 2026 10:02

This comment was marked as outdated.

@jainakanksha-msft jainakanksha-msft changed the base branch from main to akkuNodejsUpgrade June 8, 2026 06:19
@jainakanksha-msft

Copy link
Copy Markdown
Collaborator Author

@copilot resolve the merge conflicts in this pull request

Copilot AI commented Jun 8, 2026

Copy link
Copy Markdown

@copilot resolve the merge conflicts in this pull request

Merge conflicts are resolved in commit 309cfba by merging origin/akkuNodejsUpgrade into akkuSFI and fixing the conflicts in scripts/buildExe.js and package-lock.json.

@jainakanksha-msft

This comment was marked as outdated.

@jainakanksha-msft jainakanksha-msft changed the title Test Address dependency vulnerabilities: upgrade packages and fix related tests/compatibility Jun 8, 2026
@Azure Azure deleted a comment from Copilot AI Jun 8, 2026
@Azure Azure deleted a comment from Copilot AI Jun 8, 2026

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 43 out of 52 changed files in this pull request and generated 5 comments.

Comment thread src/blob/persistence/LokiBlobMetadataStore.ts
Comment thread src/blob/persistence/LokiBlobMetadataStore.ts Outdated
Comment thread src/blob/authentication/BlobSharedKeyAuthenticator.ts
Comment thread tests/blob/sas.test.ts Outdated
Comment thread package.json Outdated
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