[PR #499] [MERGED] Make STUN run by default when embedded DERP is enabled #1443

Closed
opened 2025-12-29 02:30:06 +01:00 by adam · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/juanfont/headscale/pull/499
Author: @juanfont
Created: 3/15/2022
Status: Merged
Merged: 3/18/2022
Merged by: @kradalby

Base: mainHead: mandatory-stun


📝 Commits (8)

  • b8aad54 Make STUN run by default when embedded DERP is enabled
  • d13338a Merge branch 'main' into mandatory-stun
  • 98ac88d Changed comment position
  • 8f5875e Reorg errors
  • 2e66872 Make STUN server mandatory if DERP embedded is enabled
  • d5ce7d7 Prettier
  • db9ba17 Added missing file
  • cd2914a Merge branch 'main' into mandatory-stun

📊 Changes

5 files changed (+36 additions, -29 deletions)

View changed files

📝 app.go (+14 -9)
📝 cmd/headscale/cli/utils.go (+8 -3)
📝 config-example.yaml (+4 -4)
📝 derp_server.go (+8 -10)
📝 integration_test/etc_embedded_derp/config.yaml (+2 -3)

📄 Description

This commit also allows to set an external STUN server, while running the embedded DERP server (without embedded STUN)


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/juanfont/headscale/pull/499 **Author:** [@juanfont](https://github.com/juanfont) **Created:** 3/15/2022 **Status:** ✅ Merged **Merged:** 3/18/2022 **Merged by:** [@kradalby](https://github.com/kradalby) **Base:** `main` ← **Head:** `mandatory-stun` --- ### 📝 Commits (8) - [`b8aad54`](https://github.com/juanfont/headscale/commit/b8aad5451daa24c8eeac9ab52c92b42542078675) Make STUN run by default when embedded DERP is enabled - [`d13338a`](https://github.com/juanfont/headscale/commit/d13338a9fbbe07221138850daa11a1a34283292f) Merge branch 'main' into mandatory-stun - [`98ac88d`](https://github.com/juanfont/headscale/commit/98ac88d5ef5d94fca6da47600d031f10e6833d62) Changed comment position - [`8f5875e`](https://github.com/juanfont/headscale/commit/8f5875efe4267ad859fae5e1afc21581856dc6af) Reorg errors - [`2e66872`](https://github.com/juanfont/headscale/commit/2e6687209bd3334f96e1d105375085966beda808) Make STUN server mandatory if DERP embedded is enabled - [`d5ce7d7`](https://github.com/juanfont/headscale/commit/d5ce7d752315cf31520052b67c182f25872a3ec1) Prettier - [`db9ba17`](https://github.com/juanfont/headscale/commit/db9ba17920ca3eeea306b8f82fc121403e4f3c29) Added missing file - [`cd2914a`](https://github.com/juanfont/headscale/commit/cd2914ab3bebee6aa08e7fbb45a3d4a46bc07e9d) Merge branch 'main' into mandatory-stun ### 📊 Changes **5 files changed** (+36 additions, -29 deletions) <details> <summary>View changed files</summary> 📝 `app.go` (+14 -9) 📝 `cmd/headscale/cli/utils.go` (+8 -3) 📝 `config-example.yaml` (+4 -4) 📝 `derp_server.go` (+8 -10) 📝 `integration_test/etc_embedded_derp/config.yaml` (+2 -3) </details> ### 📄 Description This commit also allows to set an external STUN server, while running the embedded DERP server (without embedded STUN) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
adam added the pull-request label 2025-12-29 02:30:06 +01:00
adam closed this issue 2025-12-29 02:30:06 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/headscale#1443