-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rework hostname handling #560
Merged
Merged
Changes from 1 commit
Commits
Show all changes
44 commits
Select commit
Hold shift + click to select a range
663e838
Nickname support
bravechamp 9901d6b
Ability to clear nickname
bravechamp 60ee046
Normalize nickname before saving to database
bravechamp c8aa653
Merge branch 'main' into main
kradalby 8845938
Merge branch 'main' into main
kradalby 6e27680
Rename name -> hostname, nickname -> givenname
kradalby caf79f6
Change nickname to givenname in proto
kradalby 6b79679
Generate from proto
kradalby 14994cb
Use new logic and fields for dns
kradalby 62808cb
Bubble error up to user for rename
kradalby 06c928b
Migrate name and nickname fields
kradalby 79704dc
Update command with new fields
kradalby 7ef8cd8
Fix comment
kradalby 8504d0d
Move todo to correct file
kradalby f4873d9
Fix rename cli error
kradalby 177c21b
Add helper function to create a unique givenname
kradalby 03cccd6
Reword FQDN normalize errors to not _only_ cover namespaces
kradalby 5fa3016
Generate unique givennames for hosts joining (and debug added)
kradalby 9b393eb
Add integration cli tests for rename command
kradalby 4aae917
Require GivenName to be unique
kradalby 163e5c2
fix trace log message
kradalby e631c6f
Merge master
kradalby 9ebeb3d
Retreive hostnames from headscale, now that they are random
kradalby 802eb93
Make sure givenname is set for preauthkeys
kradalby 4a9d3be
Use new names to resolve magic dns
kradalby 77ceeaf
Test magic dns with the correct urls
kradalby 5fa9875
move populate to after when given_name exist
kradalby 4f3f054
Fix some issues in testing with new hostname handling
juanfont ef497ca
Merge pull request #2 from juanfont/fix-rename-integration-tests
kradalby 6eac504
Merge branch 'main' into rename-fixess
kradalby fc502e1
Update golines and fix go mod checksum
kradalby 59a1a85
Change to a go generics set implementation, no more casting :tada:
kradalby 5316dd9
Use new nix stable (22.05)
kradalby 3a3fc0a
Update headscale checksum
kradalby 9993f51
docs(README): update contributors
github-actions[bot] dbc1d98
Revert golines
kradalby a992840
Give UpdateMachine a more meaningful name
kradalby a443255
Validate isOutdated against all namespaces
kradalby 4ffd3ea
Override golangci-lint to use go 1.17
kradalby 266aac9
Update CHANGELOG
kradalby d11279e
Merge branch 'main' into rename-fixess
kradalby 9175aca
Merge branch 'main' into rename-fixess
kradalby f7edea5
Merge branch 'main' into rename-fixess
kradalby 7f7cd73
Merge branch 'main' into rename-fixess
kradalby File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
You are viewing a condensed version of this merge commit. You can view the full changes here.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👀