tailscale / tailscale/tailscale
Default ACL text should show all features
- 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
Assessment
This issue has not been assessed yet.