[PR #2569] [MERGED] Remove subnet router visibility workaround from docs #2738

Closed
opened 2025-12-29 04:18:42 +01:00 by adam · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/juanfont/headscale/pull/2569
Author: @nblock
Created: 5/5/2025
Status: Merged
Merged: 5/5/2025
Merged by: @nblock

Base: mainHead: router-no-access


📝 Commits (1)

  • 4513c2b Remove subnet router visibility workaround from docs

📊 Changes

2 files changed (+6 additions, -16 deletions)

View changed files

📝 docs/ref/acls.md (+2 -4)
📝 docs/ref/routes.md (+4 -12)

📄 Description

Previous Headscale versions required a dedicated rule to make a subnet router visible to clients. This workaround is no longer required.

  • have read the CONTRIBUTING.md file
  • raised a GitHub issue or discussed it on the projects chat beforehand
  • added unit tests
  • added integration tests
  • updated documentation if needed
  • updated CHANGELOG.md

🔄 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/2569 **Author:** [@nblock](https://github.com/nblock) **Created:** 5/5/2025 **Status:** ✅ Merged **Merged:** 5/5/2025 **Merged by:** [@nblock](https://github.com/nblock) **Base:** `main` ← **Head:** `router-no-access` --- ### 📝 Commits (1) - [`4513c2b`](https://github.com/juanfont/headscale/commit/4513c2bde0448c8f6eddbf14290d7b7acb73b867) Remove subnet router visibility workaround from docs ### 📊 Changes **2 files changed** (+6 additions, -16 deletions) <details> <summary>View changed files</summary> 📝 `docs/ref/acls.md` (+2 -4) 📝 `docs/ref/routes.md` (+4 -12) </details> ### 📄 Description Previous Headscale versions required a dedicated rule to make a subnet router visible to clients. This workaround is no longer required. - [ ] have read the [CONTRIBUTING.md](./CONTRIBUTING.md) file - [ ] raised a GitHub issue or discussed it on the projects chat beforehand - [ ] added unit tests - [ ] added integration tests - [x] updated documentation if needed - [ ] updated CHANGELOG.md --- <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 04:18:42 +01:00
adam closed this issue 2025-12-29 04:18:42 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/headscale#2738