Skip to content

feat: peer wireguard#2445

Merged
coadler merged 11 commits into
mainfrom
colin/wireguard-peer
Jun 24, 2022
Merged

feat: peer wireguard#2445
coadler merged 11 commits into
mainfrom
colin/wireguard-peer

Conversation

@coadler
Copy link
Copy Markdown
Contributor

@coadler coadler commented Jun 16, 2022

Adds support for port-forwarding over Wireguard instead of WebRTC.

coder wireguard-tunnel <workspace> --tcp 8080:8080

@coadler coadler force-pushed the colin/wireguard-peer branch 2 times, most recently from 28f96a4 to 41573c7 Compare June 22, 2022 18:09
@coadler coadler force-pushed the colin/wireguard-peer branch from 41573c7 to 2361e15 Compare June 22, 2022 18:23
@coadler coadler marked this pull request as ready for review June 22, 2022 18:41
@coadler coadler requested a review from a team as a code owner June 22, 2022 18:41
// From codersdk/workspaceagents.go:47:6
// Named type "tailscale.com/types/key.NodePublic" unknown, using "any"
// eslint-disable-next-line @typescript-eslint/no-explicit-any
readonly wireguard_public_key: any
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

any is unfortunate but I'm guessing it can't be avoided?

Copy link
Copy Markdown
Contributor Author

@coadler coadler Jun 22, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm unsure, maybe @Emyrk knows? The frontend won't ever need these fields anyways.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay, no worries then!

Copy link
Copy Markdown
Contributor

@presleyp presleyp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

frontend ✅

Comment thread cli/wireguardtunnel.go
Comment thread coderd/provisionerdaemons.go Outdated
Comment thread cli/wireguardtunnel.go
Comment thread peer/peerwg/wireguard.go Outdated
Comment thread peer/peerwg/wireguard.go Outdated
Comment thread peer/peerwg/wireguard.go
Comment thread peer/peerwg/wireguard.go Outdated
Comment thread agent/wireguard.go Outdated
Comment thread peer/peerwg/derp.go
Comment thread peer/peerwg/peermessage.go Outdated
Comment thread peer/peerwg/wireguard.go
Comment thread peer/peerwg/wireguard.go Outdated
Comment thread peer/peerwg/wireguard.go Outdated
Comment thread peer/peerwg/wireguard.go Outdated
Comment thread peer/peerwg/wireguard.go Outdated
@coadler coadler requested a review from kylecarbs June 23, 2022 21:58
Copy link
Copy Markdown
Member

@kylecarbs kylecarbs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tests shall be added later today as we refactor this API to work with SSH and our other protocols. This is a hidden feature for right now, and not visible or default for anyone!

@coadler coadler merged commit 05b67ab into main Jun 24, 2022
@coadler coadler deleted the colin/wireguard-peer branch June 24, 2022 15:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants