-
Notifications
You must be signed in to change notification settings - Fork 5.9k
Add UI around ports #1944
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
Yea some UI around here to control which ports can be proxied and with or without auth would be really nice. |
@nhooyr Has there been any progress on this? If not, where would the relevant code be so that I may take a peek? |
We haven't done anything here yet. Authentication for the path proxy: Authentication for the domain proxy: A command line flag feels like an unfortunate way to implement this to me (definitely easy though) so in my mind we'd add to our |
@code-asher I agree with you! However, I think it's still better to have a stopgap |
I'm on the fence; I would definitely rather not have to maintain a feature we only added as a stop-gap. 😄 I'm also quite wary of a feature that disables auth entirely for the proxy since that defeats having auth at all (you can proxy code-server itself, for example). |
We could have the flag take port numbers instead but personally I feel doing it properly from the get-go is the right answer. I'll go ahead and add this to the next milestone; if someone doesn't take it in the next week or so I'll put it on my todo list. It'd be great to have our own built-in plugin anyway for adding other features into VS Code that we've been wanting. |
Actually proxying to code-server wouldn't let you in, you'd just get the login page...not sure what I was thinking there. I do think it's sketchy though since you can access any port on the machine without authenticating which sounds like a security nightmare. 😆 |
@code-asher I'm running a multi-user VSCode setup (using code-server on Docker Swarm) sitting behind Traefik (a load balancer), so this is where I'm coming from. Right now I manually create a rule for say, |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no activity occurs in the next 5 days. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no activity occurs in the next 5 days. |
My apologies if there's already a way to do this, but it looks like there isn't.
For me there's really no use case to secure the proxied ports, but I do like code-server running behind authentication wall so I don't want to disable the whole thing. It'd be great if there was an option like
code-server --disable-proxy-auth
that would allow/proxy/<port>
requests to go through.The text was updated successfully, but these errors were encountered: