Skip to content
This repository was archived by the owner on Nov 12, 2026. It is now read-only.

Tailscale app: say when a VPN's passwords aren't saved - #57

Merged
techmore merged 1 commit into
mainfrom
claude/tailscale-vpn-passwords
Oct 29, 2026
Merged

techmore merged 1 commit into
mainfrom
claude/tailscale-vpn-passwords

Conversation

@techmore

Copy link
Copy Markdown
Owner

Owner report: the VPN "didn't save my passwords".

Cause: BFS (L2TP/IPsec) has password-flags = 2 and ipsec-psk-flags = 2, which means "ask every time, never store". This desktop also runs no NetworkManager secret agent, so nmcli connection up from the app fails with "No agents were available" (NetworkManager log, 09:49).

  • The VPN card reads the *-flags entries from vpn.data (flags only, no secrets). When a secret is never stored, it shows a warning and a Save Passwords… button that opens nm-connection-editor --edit <uuid>.
  • A connect that fails for missing secrets says so in a toast and opens the same editor.
  • No secrets are read or written by the app. The owner stores them in the editor ("Store the password for all users").

Checked: tests (new flag-parsing cases), clippy clean, fixture render, and a render against the real BFS connection (warning shown). Installed and restarted locally.

🤖 Generated with Claude Code

… them

BFS stores neither its password nor its IPsec key (flags 2, ask every time) and
the desktop has no NetworkManager secret agent, so connecting from the app
failed with "No agents were available". The VPN card now says so and opens
nm-connection-editor for that connection; a connect that fails for missing
secrets says why and opens it too.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@techmore
techmore merged commit 86550f6 into main Oct 29, 2026
1 check passed
@techmore
techmore deleted the claude/tailscale-vpn-passwords branch October 29, 2026 13:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant