You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
[Feat]: make VSCODE_PROXY_URI use the subdomain proxy when it is enabled #5311
When VSCODE_PROXY_URI is enabled, use the subdomain proxy.
Why do you want this feature?
Popular extensions like Tabnine can't use relative paths and need to be able to talk to code-server on specific ports/paths in order to work correctly.
Are there any workarounds to get this functionality today?
We might also want a way to override this for cases like Coder where we already provide a subdomain proxy outside of code-server. For this we can probably just check if that variable is already set and if so avoid overriding.
To implement we need to check the proxy-domain flag and use that in the environment variable. It can be defined multiple times so maybe we just use the first one. So more or less I think it would be {{port}}.${args["proxy-domain"][0]}. If the flag is not set we just keep using the path-based proxy.
I also think we should go ahead and patch asExternalUri to use this same environment variable although we should use the other ticket for that (and a separate PR).
Activity
code-asher commentedon Jun 30, 2022
We might also want a way to override this for cases like Coder where we already provide a subdomain proxy outside of code-server. For this we can probably just check if that variable is already set and if so avoid overriding.
To implement we need to check the
proxy-domainflag and use that in the environment variable. It can be defined multiple times so maybe we just use the first one. So more or less I think it would be{{port}}.${args["proxy-domain"][0]}. If the flag is not set we just keep using the path-based proxy.I also think we should go ahead and patch
asExternalUrito use this same environment variable although we should use the other ticket for that (and a separate PR).