-
Notifications
You must be signed in to change notification settings - Fork 5.9k
Document proxy and shortcomings #4026
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
Comments
Very sorry to hear you spent your whole day researching this 😢
We have pretty thorough guides for deploying code-server which is mentioned in the
Where did you look for this information and what was missing? That will help us make a note to improve it!
I don't do any Vue development, but if you provide an example or repro steps, I'd be happy to update that example to show one with a subdirectory too!
We can definitely add more docs around how the proxy works! Let me know what you'd like to see, I'll add and then add you as a reviewer!
I'm not familiar with this and not sure how difficult it would be to support but if you open a separate feature request, we can discuss and see what's possible. |
It's the documentation here: https://coder.com/docs/code-server/latest/guide#stripping-proxyport-from-the-request-path
Actually, I found some strange bugs for my Vue app where the trailing slash missing would cause path 404 issue. So there might be some bug or misconfiguration on my side. After cleaning caches and review the configurations from documentation, the
That might be the only usage or description for I think those information were missing for those who were trying to build a Cloud IDE on their own.
I'd like to help you out with that. Actually, I just joined the Slack channel yesterday.
Yes sure, please! I would like to improve the documentation as well, with the vue app deployment guidance.
So, with the new documentation said, we can add a property called sockPath to specify a path for sockjs-node, in fact, the definination of this property was missing from the documentation of vue/cli3. |
Here's a short question. By using Is this attempted to be intentional? |
This issue stated out some implementation issue, webpack/webpack-dev-server#2535 |
Interesting...this would be a great question for @code-asher |
I'll defer to you. I don't use Vue so can't chime in really 😢 |
That's awesome
Do you want to take a look #4033 and then open a PR to add anything that's missing? |
@bpmct shared this for getting it to work with an external domain: |
Yeah the path-based proxy has all those shortcomings. The
subdomain-based proxy is definitely superior. Seems like a
good idea to mention that in the docs.
|
Thanks for that. I will test it out to see if that works. |
Can we open up another PR to detail this information in documentation? I think it would be useful for newbie developers. |
Definitely! I'll change this issue to reflect that. |
I will provide a |
Both subdomain and path-based proxy |
I have spent my whole day wo research and find out how I can deploy a Cloud IDE instance for vue development environment. I found out that
/absproxy
works very differently from/proxy
which the documentation didn't mentioned about a lot.Another problem was the
sockjs-node
, I have looked up these issues and prs to find out more clues:Infact,
create-react-app
script did fixed thesockjs-node
's always pointing root/
issue, meanwhile, there's no any effective or useful way to makevue/cli3
''ssockjs-node
to work.There is a example that worth vue developers to take a look:
In this example, the demo was shown as local development envirnment with no subdirectory, however, it was painful to setup a nginx/Apache reverse proxy server rewrite rule without reading the proxy code of code-server it self (which took the most of my time).
Will there be any documentation to summarize the way those proxy work, and maybe, support vue/sockjs-node proxy in HTTPS development envirnment (HTTPS development envirnment is very important in this age of time)?
The text was updated successfully, but these errors were encountered: