fix(manage): vpn revoke refuses ambiguous names, can delete stale nodes, verifies result #259

Merged

jfraeys merged 1 commit from jfraeys/fix/vpn-revoke-ambiguity into main

2026-10-16 22:27:55 -04:00

jfraeys commented

2026-10-16 22:22:36 -04:00

Owner

A name matching several nodes (tailnet today has two web, ids 1 live and 63 stale) is refused with the ids; it used to take the first match.
--delete removes a stale record (headscale nodes delete --identifier ID --force) and re-lists to confirm it is gone.
A failing headscale command prints its output (it was sent to /dev/null).
New test with a stubbed Headscale, wired into lint.

Not verified against live Headscale: the read was blocked, so why nodes delete --identifier 21 --force did nothing is still unknown; the new output will show it.

🤖 Generated with Claude Code

https://claude.ai/code/session_01Ge5tovcwdEUBtPwUktsZZZ

- A name matching several nodes (tailnet today has two `web`, ids 1 live and 63 stale) is refused with the ids; it used to take the first match. - `--delete` removes a stale record (`headscale nodes delete --identifier ID --force`) and re-lists to confirm it is gone. - A failing headscale command prints its output (it was sent to /dev/null). - New test with a stubbed Headscale, wired into lint. Not verified against live Headscale: the read was blocked, so why `nodes delete --identifier 21 --force` did nothing is still unknown; the new output will show it. 🤖 Generated with [Claude Code](https://claude.com/claude-code) https://claude.ai/code/session_01Ge5tovcwdEUBtPwUktsZZZ

jfraeys added 1 commit

2026-10-16 22:22:36 -04:00

fix(manage): vpn revoke refuses ambiguous names, can delete stale nodes, verifies the result

Lint / lint (pull_request) Successful in 3m7s

Details

Secret scan / scan (pull_request) Successful in 18s

Details

f41a444397

A name shared by two nodes (live web + a stale web) resolved to the first match and could
expire the live node. --delete removes stale records and checks they are gone; a failing
headscale command now shows its output instead of discarding it.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Ge5tovcwdEUBtPwUktsZZZ

jfraeys merged commit 9dede7b5bb into main

2026-10-16 22:27:55 -04:00

jfraeys referenced this pull request from a commit

2026-10-16 22:27:55 -04:00

Merge pull request 'fix(manage): vpn revoke refuses ambiguous names, can delete stale nodes, verifies result' (#259) from jfraeys/fix/vpn-revoke-ambiguity into main

jfraeys referenced this pull request

2026-10-17 14:14:41 -04:00

Migrate production compute from Linode to OVH (starting with services) #187