tailscale / tailscale/tailscale

Default ACL text should show all features

Open
#2,148 2 comments 0 reactions 0 assignees View on GitHub
acl acl-test console L5 All users P1 Nuisance T4 Docs
Dominant language
Go
Stars
36.5k
Forks
3.2k
Avg merge
2d 3h
Merged PRs (30d)
123

Description

We haven't updated our default ACL text in a long time. We should make sure it shows or references all the interesting features:
- [ ] Hostnames and subnets (both as source and as destination)
- [ ] autogroup:same-user (or whatever we end up calling it)
- [ ] role:user and role:admin (or whatever the new names are)
- [ ] ACL tags and TagOwners
- [ ] Users as destinations, not just sources
- [ ] Tests
- [ ] Route AutoApprovers
- [x] SSH

It would be okay to link to KB articles etc from the comments to explain the various features.

Updating the default text is generally harmless since the text won't actually save in the database unless it's modified.

Note that the default/example ACLs must always have a global "allow everything" rule at the bottom, since the default is to allow everything.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.