-
Notifications
You must be signed in to change notification settings - Fork 452
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
base url support #320
Comments
+1 on this I came here exactly looking for this. |
@chartera @ffleandro The purpose of this project is to do the minimal changes on upstream code, for this feature request you should open an issue in upstream repo https://github.com/microsoft/vscode/issues |
I did get it working for the most part but I'm honestly not sure if that commit has all of the needed changes. A lot of what I did was hacking on the minified js then finding the relevant bits to change in the source. I had planned to figure out exactly what needed to be changed and send the appropriate pull requests once I had I working, but became distracted with other work. I will get back to it and put together a pull request (hopefully) by next week. It looks like the changes could be easily made upstream, but if they aren't receptive I think the maintainers here should consider implementing something. It would be a pretty minor patch and it would be more value to the users of |
For what it's worth I did spend some time trying to get this done last week, but ran out of time. Turns out doing things right takes more time than just making something work for my own purposes... I should have more time in a day or two to work on it again. |
I've seen some commits recently microsoft#147311 microsoft#148707 that I think will enable this in the next release. |
I've been trying to work around this by installing nginx in the Docker image, but I'm struggling to get it to work with some rewrites or redirects as a temporary workaround. Anyone got that to work? |
What if that path dynamically changes depending on the user and other things? This looks to me as if the path must be fixed. |
I also stumbled across this problem... maybe one should mention that there is also a Visual Studio Code distribution code-server, which has patches applied (see also these search results) that make Visual Studio Code accessible on arbitrary base URLs, i.e. behind reverse proxies such as jupyter-server-proxy. I think, I will be using that one for now. |
Yes, this is definitely an option. At the moment this does not work yet for other reasons: |
@jhgoebbert @kathoef running code-server behind the jupyter-server-proxy works - but did you have some other aspect of the functionality in mind that is currently missing? |
@rokroskar Perhaps I was not specific enough, At least until end of June it was not possible to run a code-server behind a JupyterHub + jupyter-server-proxy because it did not like the use of a configurable https proxy which modifies the base url: #320 (comment) |
@jhgoebbert if you're talking about code-server (i.e. as per the comment by @kathoef above) then yes - we've been using it for a while now and it mostly works. Often the jupyter/python extension breaks because of some version incompatibility which requires us to do some maintenance. We use the little jupyter-server-extension wrapper: https://github.com/betatim/vscode-binder. You can find the simple script we use here. You can see an example project here. We would prefer to use vscode directly to avoid issues with extensions, which is why I'm following this thread ;) |
Thank you @rokroskar |
Upstream added support for |
@mering you can use the openvscode-server-insiders release or wait until the next stable release which should be released early april |
closing as it got implemented upstream |
As I noted in microsoft#202491 (comment), this does not add the necessary functionality to work behind a proxy which this ticket is about. |
@jeanp413 can you please re-open as the requested feature is not implemented upstream. Only some other functionality which could allow a smaller downstream patch. |
This issue is about base path support which the upstream PR added, any additional feature request should be done in the upstream repo as noted in my previous comment #320 (comment) |
Quoting the issue description:
I tried to achieve exactly this (proxy which redirects @jeanp413 so can you please re-open this issue? |
in case, using dynamic port expose with docker container, using reverse proxy by nginx |
I created microsoft#210399. |
It would be great to use it at another HTTP path, instead of /.
I wanna run it behind proxy and not setup a separate sudomain for it.
The text was updated successfully, but these errors were encountered: