Skip to content

Encode requests to npm.antfu.dev#13350

Merged
delucis merged 1 commit intomainfrom
chris/npm-workaround
Mar 6, 2026
Merged

Encode requests to npm.antfu.dev#13350
delucis merged 1 commit intomainfrom
chris/npm-workaround

Conversation

@delucis
Copy link
Copy Markdown
Member

@delucis delucis commented Mar 6, 2026

Description (required)

This PR is a workaround for antfu/fast-npm-meta#33

antfu/fast-npm-meta#32 caused our requests for package data from https://npm.antfu.dev to fail. Until the issue is fixed, we can encode our requested packages to work around the change in behaviour.

@netlify
Copy link
Copy Markdown

netlify bot commented Mar 6, 2026

Deploy Preview for astro-docs-2 ready!

Name Link
🔨 Latest commit 8ff9a00
🔍 Latest deploy log https://app.netlify.com/projects/astro-docs-2/deploys/69aab43a6e3cc7000738645c
😎 Deploy Preview https://deploy-preview-13350--astro-docs-2.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Copy Markdown
Member

@ArmandPhilippot ArmandPhilippot left a comment

Choose a reason for hiding this comment

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

Looking at the issue, this changes makes sense to me!

@delucis
Copy link
Copy Markdown
Member Author

delucis commented Mar 6, 2026

Thanks @ArmandPhilippot! I’ll merge to unblock things for everyone. We can always revert if they fix the issue.

@delucis delucis merged commit 47ed41a into main Mar 6, 2026
11 checks passed
@delucis delucis deleted the chris/npm-workaround branch March 6, 2026 11:15
@alex-key
Copy link
Copy Markdown

alex-key commented Mar 6, 2026

After antfu/fast-npm-meta#34 is merged both encoded "+" (%2B) and raw "+" should work ok for batch request.

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