Skip to content

android: un-prepare VPN service on revocation#748

Merged
kari-ts merged 1 commit into
mainfrom
kari/task
Mar 2, 2026
Merged

android: un-prepare VPN service on revocation#748
kari-ts merged 1 commit into
mainfrom
kari/task

Conversation

@kari-ts

@kari-ts kari-ts commented Feb 27, 2026

Copy link
Copy Markdown
Collaborator

vpnPrepared tracks whether Tailscale holds VPN permission. When permission is revoked, the VPNService must be reestablished. If vpnPrepared stays true, the Go backend reconnects, but startVPN() is not called in the ConnectView and the tunnel is not reestablished

Updates tailscale/tailscale#17190

vpnPrepared tracks whether Tailscale holds VPN permission. When permission is revoked, the VPNService must be reestablished. If vpnPrepared stays true, the Go backend reconnects, but startVPN() is not called in the ConnectView and the tunnel is not reestablished

Updates tailscale/tailscale#17190
Signed-off-by: kari-ts <kari@tailscale.com>
@kari-ts
kari-ts requested a review from barnstar February 27, 2026 05:22
kari-ts added a commit that referenced this pull request Feb 28, 2026
QuickToggleService was not checking whether or not Tailscale had VPN permission, resulting in a bug where if VPN permission had been revoked, Tailscale would not start properly. This fixes it by maybe launching MainActivity, which gets permission. This is dependent on #748, which sets vpnPrepared on revocation.

Updates tailscale/tailscale#17190

Signed-off-by: kari-ts <kari@tailscale.com>
@kari-ts
kari-ts merged commit 50154cb into main Mar 2, 2026
4 checks passed
@kari-ts
kari-ts deleted the kari/task branch March 2, 2026 17:42
kari-ts added a commit that referenced this pull request Mar 2, 2026
#749)

QuickToggleService was not checking whether or not Tailscale had VPN permission, resulting in a bug where if VPN permission had been revoked, Tailscale would not start properly. This fixes it by maybe launching MainActivity, which gets permission. This is dependent on #748, which sets vpnPrepared on revocation.

Updates tailscale/tailscale#17190

Signed-off-by: kari-ts <kari@tailscale.com>
nadeemakhter0602 pushed a commit to nadeemakhter0602/TailMon that referenced this pull request Mar 22, 2026
vpnPrepared tracks whether Tailscale holds VPN permission. When permission is revoked, the VPNService must be reestablished. If vpnPrepared stays true, the Go backend reconnects, but startVPN() is not called in the ConnectView and the tunnel is not reestablished

Updates tailscale/tailscale#17190

Signed-off-by: kari-ts <kari@tailscale.com>
nadeemakhter0602 pushed a commit to nadeemakhter0602/TailMon that referenced this pull request Mar 22, 2026
tailscale#749)

QuickToggleService was not checking whether or not Tailscale had VPN permission, resulting in a bug where if VPN permission had been revoked, Tailscale would not start properly. This fixes it by maybe launching MainActivity, which gets permission. This is dependent on tailscale#748, which sets vpnPrepared on revocation.

Updates tailscale/tailscale#17190

Signed-off-by: kari-ts <kari@tailscale.com>
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.

2 participants