[PR #359] [MERGED] Add YAML support to ACLs #1404

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

📋 Pull Request Information

Original PR: https://github.com/juanfont/headscale/pull/359
Author: @kradalby
Created: 2/27/2022
Status: Merged
Merged: 3/1/2022
Merged by: @kradalby

Base: mainHead: yaml-acls


📝 Commits (6)

  • 67d6c8f Remove oversensitive tracing output
  • 8a3a0b6 Add YAML support to ACLs
  • c159eb7 Add basic test of yaml parsing
  • e0b9a31 Add note to config example
  • d6f6939 Update changelog
  • 1058124 Merge branch 'main' into yaml-acls

📊 Changes

7 files changed (+96 additions, -29 deletions)

View changed files

📝 CHANGELOG.md (+4 -0)
📝 acls.go (+31 -9)
📝 acls_test.go (+16 -0)
📝 acls_types.go (+33 -11)
📝 config-example.yaml (+2 -1)
tests/acls/acl_policy_basic_wildcards.yaml (+10 -0)
📝 utils.go (+0 -8)

📄 Description

This resolves #326

  • read the CONTRIBUTING guidelines
  • raised a GitHub issue or discussed it on the projects chat beforehand
  • added unit 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/359 **Author:** [@kradalby](https://github.com/kradalby) **Created:** 2/27/2022 **Status:** ✅ Merged **Merged:** 3/1/2022 **Merged by:** [@kradalby](https://github.com/kradalby) **Base:** `main` ← **Head:** `yaml-acls` --- ### 📝 Commits (6) - [`67d6c8f`](https://github.com/juanfont/headscale/commit/67d6c8f9465d52401666b27a5f96aff2bf17b5f9) Remove oversensitive tracing output - [`8a3a0b6`](https://github.com/juanfont/headscale/commit/8a3a0b64037bea8c15700943d37a16f02375cf28) Add YAML support to ACLs - [`c159eb7`](https://github.com/juanfont/headscale/commit/c159eb7541e18f3ed1c926826d5075a7bf54b5cf) Add basic test of yaml parsing - [`e0b9a31`](https://github.com/juanfont/headscale/commit/e0b9a317f4c30ad07e275b179b5037d61b8bdc02) Add note to config example - [`d6f6939`](https://github.com/juanfont/headscale/commit/d6f6939c54dc0144fa9cc5e6ddf97e45652780ae) Update changelog - [`1058124`](https://github.com/juanfont/headscale/commit/105812421efa6aedff500bbe4451ec024767c29e) Merge branch 'main' into yaml-acls ### 📊 Changes **7 files changed** (+96 additions, -29 deletions) <details> <summary>View changed files</summary> 📝 `CHANGELOG.md` (+4 -0) 📝 `acls.go` (+31 -9) 📝 `acls_test.go` (+16 -0) 📝 `acls_types.go` (+33 -11) 📝 `config-example.yaml` (+2 -1) ➕ `tests/acls/acl_policy_basic_wildcards.yaml` (+10 -0) 📝 `utils.go` (+0 -8) </details> ### 📄 Description This resolves #326 <!-- Please tick if the following things apply. You… --> - [x] read the [CONTRIBUTING guidelines](README.md#user-content-contributing) - [x] raised a GitHub issue or discussed it on the projects chat beforehand - [x] added unit tests - [x] updated documentation if needed - [x] updated CHANGELOG.md <!-- If applicable, please reference the issue using `Fixes #XXX` and add tests to cover your new code. --> --- <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:29:57 +01:00
adam closed this issue 2025-12-29 02:29:57 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/headscale#1404