Skip to content

fix(spanner): override gcp-metadata to resolve punycode deprecation#7815

Merged
surbhigarg92 merged 2 commits intomainfrom
test-pr-2
Mar 23, 2026
Merged

fix(spanner): override gcp-metadata to resolve punycode deprecation#7815
surbhigarg92 merged 2 commits intomainfrom
test-pr-2

Conversation

@alkatrivedi
Copy link
Copy Markdown
Contributor

@alkatrivedi alkatrivedi commented Mar 18, 2026

Description

This PR adds overrides for gcp-metadata to version ^7.0.1 to resolve Node.js deprecation warnings.

Issue

When running on Node.js 22, deep sub-dependencies (transitively imported via gcp-metadata) trigger the following warning:
DeprecationWarning: The punycode module is deprecated. Please use a userland alternative instead.

Solution

Forcing @google-cloud/spanner to resolve gcp-metadata to ^7.0.1 ensures that updated versions of internal libraries (e.g., gaxios) are fetched, correctly bypassing the deprecated punycode modules.

Changes

Added package manager overrides in package.json:

  • "overrides" for npm / yarn workspaces compatibility.
  • "pnpm.overrides" for pnpm resolving.

BEGIN_COMMIT_OVERRIDE
fix(spanner): override gcp-metadata to resolve punycode deprecation](#7815 (comment))
END_COMMIT_OVERRIDE

@alkatrivedi alkatrivedi requested a review from a team as a code owner March 18, 2026 06:58
@surbhigarg92 surbhigarg92 changed the title presubmit failure check fix(spanner): override gcp-metadata to resolve punycode deprecation Mar 18, 2026
Copy link
Copy Markdown
Contributor

@olavloite olavloite left a comment

Choose a reason for hiding this comment

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

Just to be sure: This change does not change the minimum version of Node or any other major components that are required to use this library, right?

@surbhigarg92
Copy link
Copy Markdown
Contributor

Just to be sure: This change does not change the minimum version of Node or any other major components that are required to use this library, right?

No. gcp-metadata package needs "node": ">=18" which we are already on.

@surbhigarg92 surbhigarg92 merged commit 16971e7 into main Mar 23, 2026
35 checks passed
@surbhigarg92 surbhigarg92 deleted the test-pr-2 branch March 23, 2026 08:14
@sofisl sofisl added the release-please:force-run To run release-please label Apr 2, 2026
@release-please release-please Bot removed the release-please:force-run To run release-please label Apr 2, 2026
@sofisl sofisl added the release-please:force-run To run release-please label Apr 3, 2026
@release-please release-please Bot removed the release-please:force-run To run release-please label Apr 3, 2026
@sofisl sofisl added the release-please:force-run To run release-please label Apr 3, 2026
@release-please release-please Bot removed the release-please:force-run To run release-please label Apr 3, 2026
@sofisl sofisl added the release-please:force-run To run release-please label Apr 3, 2026
@release-please release-please Bot removed the release-please:force-run To run release-please label Apr 3, 2026
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.

4 participants