[PR #1673] [MERGED] node selfupdate and fix subnet router when ACL is enabled #2246

Closed
opened 2025-12-29 03:20:22 +01:00 by adam · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/juanfont/headscale/pull/1673
Author: @kradalby
Created: 1/3/2024
Status: Merged
Merged: 1/18/2024
Merged by: @kradalby

Base: mainHead: 1604-acl-subnet-routers


📝 Commits (1)

  • 8e0d97d Update ACL when node advertise route

📊 Changes

9 files changed (+506 additions, -0 deletions)

View changed files

.github/workflows/test-integration-v2-TestSubnetRouteACL.yaml (+67 -0)
📝 hscontrol/db/node.go (+13 -0)
📝 hscontrol/mapper/mapper.go (+12 -0)
📝 hscontrol/policy/acls.go (+15 -0)
📝 hscontrol/policy/acls_test.go (+75 -0)
📝 hscontrol/poll.go (+25 -0)
📝 integration/route_test.go (+272 -0)
📝 integration/tailscale.go (+2 -0)
📝 integration/tsic/tsic.go (+25 -0)

📄 Description

Fixes #1604


🔄 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/1673 **Author:** [@kradalby](https://github.com/kradalby) **Created:** 1/3/2024 **Status:** ✅ Merged **Merged:** 1/18/2024 **Merged by:** [@kradalby](https://github.com/kradalby) **Base:** `main` ← **Head:** `1604-acl-subnet-routers` --- ### 📝 Commits (1) - [`8e0d97d`](https://github.com/juanfont/headscale/commit/8e0d97dd59e5564166f34f1703809b3291a05d4b) Update ACL when node advertise route ### 📊 Changes **9 files changed** (+506 additions, -0 deletions) <details> <summary>View changed files</summary> ➕ `.github/workflows/test-integration-v2-TestSubnetRouteACL.yaml` (+67 -0) 📝 `hscontrol/db/node.go` (+13 -0) 📝 `hscontrol/mapper/mapper.go` (+12 -0) 📝 `hscontrol/policy/acls.go` (+15 -0) 📝 `hscontrol/policy/acls_test.go` (+75 -0) 📝 `hscontrol/poll.go` (+25 -0) 📝 `integration/route_test.go` (+272 -0) 📝 `integration/tailscale.go` (+2 -0) 📝 `integration/tsic/tsic.go` (+25 -0) </details> ### 📄 Description Fixes #1604 --- <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 03:20:22 +01:00
adam closed this issue 2025-12-29 03:20:22 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/headscale#2246