Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Simple reverse proxy for China users #795

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

MoRanYue
Copy link

@MoRanYue MoRanYue commented Jun 19, 2024

It added a option for Chinese, because of slow and unstable connections to GitHub and HuggingFace, it allows users to install custom nodes and models by free reverse proxies, now they are "https://mirror.ghproxy.com" for GitHub and "https://hf-mirror.com" for HuggingFace. You can choose whether using both of them, one of them, or do not use them at all. For others who are in other countries, this option SHOULD KEEP "NONE" in order to avoid possible connection problems.

Options are on the left of the Manager Dialog

@MoRanYue MoRanYue changed the title feat: Simple reserve proxy for China users feat: Simple reverse proxy for China users Jun 19, 2024
@ltdrdata
Copy link
Owner

This PR should be considered after CNR has been merged.

@MoRanYue
Copy link
Author

i think this can be improved, such as Auto-Ping to avoid clicking by mistake

@cenfun
Copy link
Contributor

cenfun commented Jul 3, 2024

@ltdrdata It seems that you already created a branch feat/cnr for CNR, is it ready to release? or is there anything that needs help on custom nodes manager UI?

@ltdrdata
Copy link
Owner

ltdrdata commented Jul 3, 2024

@ltdrdata It seems that you already created a branch feat/cnr for CNR, is it ready to release? or is there anything that needs help on custom nodes manager UI?

That branch is the WIP branch. Although it hasn't been pushed, various modifications are continuously being made.

@hben35096
Copy link

hben35096 commented Jul 21, 2024

Coincidentally, I still think manually filling in the configuration file is a better approach. Mirror sites might stop operating one day, and at that time, we can switch to another URL. Additionally, different countries might use different mirror sites.

#893

@Harry-675
Copy link

Seems this fix is not merged?

@ltdrdata
Copy link
Owner

ltdrdata commented Aug 9, 2024

Seems this fix is not merged?

This is postponed due to upcoming update.
This patch should be merged after that.

The currently ongoing update work is a very large update, and this PR cannot be easily incorporated into that update.

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.

5 participants