-
Notifications
You must be signed in to change notification settings - Fork 937
feat: Add Tailscale networking #3505
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
Conversation
This makes for a simpler setup, and reduces the likelihood a user runs into a strange issue.
@bpmct I think this is related to websockets being broken for the dev tunnel. Fixing this morning! |
@bpmct that issue you encountered with WebSockets has been fixed! |
Did some tests and confirmed #3316 and the terminal hanging is resolved, the tunnel also works. 🎉 The first time I tried it, I did have to manually reload the pop-up window in order to get things to load, however. I have not been able to reproduce sense. Maybe it was a one-off error? final_630e1f4ccd265a00839746cf_4775.mp4I also see somewhat surprising logs, although things seem stable during code-server usage coder server logs
|
Contributes to #2779. This adds Tailscale networking in addition to WebRTC. It is disabled by default, clients must opt-in (and it's hidden). The rollout plan is:
The delta should be -3,000 lines of code 😎... not to mention so so so many bugs fixed.
Todo
agent.TailscaleConn.Ping()
work!HTTPForTests
in the Tailscale fork toForceHTTP
.Might do