[PR #24678] refactor: relocate China npm registry config to base image #30742

Closed
opened 2026-02-21 20:48:08 -05:00 by yindo · 0 comments
Owner

Original Pull Request: https://github.com/langgenius/dify/pull/24678

State: closed
Merged: Yes


Summary

This pull request updates the web/Dockerfile to improve support for developers located in China by providing instructions for using faster package registries. The main changes involve adding commented lines to help speed up dependency installation by switching to local mirrors.

Improvements for developers in China:

  • Added a commented instruction to use the Taobao npm registry for faster npm installs (npm config set registry https://registry.npmmirror.com).
  • Removed the commented instruction for using the Taobao pnpm registry during installation, streamlining the Dockerfile and avoiding redundancy.

Screenshots

Before After
... ...

Checklist

  • This change requires a documentation update, included: Dify Document
  • I understand that this PR may be closed in case there was no previous discussion or issues. (This doesn't apply to typos!)
  • I've added a test for each change that was introduced, and I tried as much as possible to make a single atomic change.
  • I've updated the documentation accordingly.
  • I ran dev/reformat(backend) and cd web && npx lint-staged(frontend) to appease the lint gods
**Original Pull Request:** https://github.com/langgenius/dify/pull/24678 **State:** closed **Merged:** Yes --- ## Summary This pull request updates the `web/Dockerfile` to improve support for developers located in China by providing instructions for using faster package registries. The main changes involve adding commented lines to help speed up dependency installation by switching to local mirrors. Improvements for developers in China: * Added a commented instruction to use the Taobao npm registry for faster npm installs (`npm config set registry https://registry.npmmirror.com`). * Removed the commented instruction for using the Taobao pnpm registry during installation, streamlining the Dockerfile and avoiding redundancy. ## Screenshots | Before | After | |--------|-------| | ... | ... | ## Checklist - [ ] This change requires a documentation update, included: [Dify Document](https://github.com/langgenius/dify-docs) - [x] I understand that this PR may be closed in case there was no previous discussion or issues. (This doesn't apply to typos!) - [x] I've added a test for each change that was introduced, and I tried as much as possible to make a single atomic change. - [x] I've updated the documentation accordingly. - [x] I ran `dev/reformat`(backend) and `cd web && npx lint-staged`(frontend) to appease the lint gods
yindo added the pull-request label 2026-02-21 20:48:08 -05:00
yindo closed this issue 2026-02-21 20:48:08 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langgenius/dify#30742