chore(connlib): remove STUN-only functionality

[thomaseizinger]

With this PR, we ignore all "stun" relays sent from the portal.

Follow-up from: #3353.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment

Name	Status	Preview	Comments	Updated (UTC)
firezone	⬜️ Ignored (Inspect)	Visit Preview		Mar 16, 2024 5:38am

[github-actions]

Terraform Cloud Plan Output

Plan: 8 to add, 7 to change, 8 to destroy.

Terraform Cloud Plan

[thomaseizinger]

With this PR, we ignore or "stun" relays sent from the portal.

cc @AndrewDryga I am not sure if the portal still supports that but after this PR, we can remove it!

[github-actions]

Performance Test Results

TCP

Test Name	Received/s	Sent/s	Retransmits
direct-tcp-client2server	230.8 MiB (+0%)	232.8 MiB (+0%)	80 (-40%)
direct-tcp-server2client	240.3 MiB (+2%)	242.1 MiB (+2%)	459 (+286%)
relayed-tcp-client2server	147.8 MiB (-1%)	148.5 MiB (-1%)	155 (+5%)
relayed-tcp-server2client	150.7 MiB (+2%)	151.2 MiB (+2%)	176 (+1%)

UDP

Test Name	Total/s	Jitter	Lost
direct-udp-client2server	50.0 MiB (+0%)	0.04ms (-4%)	0.00% (NaN%)
direct-udp-server2client	50.0 MiB (+0%)	0.01ms (-29%)	0.00% (NaN%)
relayed-udp-client2server	50.0 MiB (+0%)	0.08ms (-65%)	0.00% (NaN%)
relayed-udp-server2client	50.0 MiB (-0%)	0.04ms (-19%)	0.00% (NaN%)

[thomaseizinger]

I need to think about how we want to migrate the snownet tests.

[jamilbk]

We may want to hold on merging any risky refactors while we work to stabilize things for GA. I think there's quite a bit of cleanup in the portal to remove the user facing piece of the STUN-only feature, today it's just being hidden.

[jamilbk]

I'm surprised the integration tests pass if this is a breaking API change?

[thomaseizinger]

I'm surprised the integration tests pass if this is a breaking API change?

I've left the messages there so we can still parse the STUN server but just ignore it.

[thomaseizinger]

Outdated.