Skip to content
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

#2140 Fixed reflection of hostname change #2199

Merged
merged 4 commits into from
Oct 17, 2024

Conversation

hopleus
Copy link
Contributor

@hopleus hopleus commented Oct 15, 2024

  • have read the CONTRIBUTING.md file
  • raised a GitHub issue or discussed it on the projects chat beforehand
  • added unit tests
  • added integration tests
  • updated documentation if needed
  • updated CHANGELOG.md

Fixes #2140

hscontrol/types/node.go Outdated Show resolved Hide resolved
@hopleus hopleus requested a review from kradalby October 17, 2024 11:56
@kradalby kradalby merged commit b6dc6eb into juanfont:main Oct 17, 2024
122 of 123 checks passed
@kradalby
Copy link
Collaborator

Thank you! your help is much appreciated, I'm a bit overloaded at the moment, so I try to come to it as I have some spare moments.

hopleus added a commit to hopleus/headscale that referenced this pull request Oct 25, 2024
* juanfont#2140 Fixed updating of hostname and givenName when it is updated in HostInfo

* juanfont#2140 Added integration tests

* juanfont#2140 Fix unit tests

* Changed IsAutomaticNameMode to GivenNameHasBeenChanged. Fixed errors in files according to golangci-lint rules
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] Hostname changes are not reflected
2 participants