-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
Improve --proxy-domain documentation #6897
Comments
This feature is for proxying any arbitrary port dynamically for the purposes of web development, so it has to be a number. If it was not a number you would need something to map words to the right port number. As one example, if you have code-server hosted at http://my-code-server.com and you are developing a PHP application on port 8080, you can browse It is entirely opt-in though, so you do not have to use it. You can use the path-based proxy instead (http://my-code-server.com/proxy/8080), or forward the port with SSH, or do something else. |
for proxy & reverse proxy and for cloud services it's better for it to be mapped as unix socket instead of TCP/IP socket ? why it's implemented mainly to send back responses from 8080.my-code-server.com ? . I had to do extra coding to alter source packets in between nginx and code-server to fix the white screen in the code-server . it will be a lot better to have two channels : one for my-code-server.com , the second is my-code-server.com correct me please if there is a way to have my-code-server.com only |
by default the code-server --proxy-domain must be given only the domain name . if I want to make another instance I will start another process of code-server on another port with that domain . |
You would not need a second instance, and you should not need to alter any packets.
If you add In your case, it sounds like you do not need the port proxy at all. So I would just remove |
I tried each option behind reverse proxy and no luck . I got white screen . I'll install it on a clean ubuntu machine and post steps of how to reproduce the problem . |
#6353 (comment)
The text was updated successfully, but these errors were encountered: