diff options
| author | Nick Khyl <nickk@tailscale.com> | 2025-02-07 11:18:57 -0600 |
|---|---|---|
| committer | Nick Khyl <1761190+nickkhyl@users.noreply.github.com> | 2025-02-10 14:54:14 -0600 |
| commit | 48dd4bbe21e1b4ade2ca23acce86a115cb0f59ff (patch) | |
| tree | 6a94d649adc854b8820c7cd87822257ae7297d46 /control/controlhttp/controlhttpserver/controlhttpserver.go | |
| parent | 11cd98fab0b1f11fbad88ff6cb3e730c9cdc7245 (diff) | |
| download | tailscale-48dd4bbe21e1b4ade2ca23acce86a115cb0f59ff.tar.xz tailscale-48dd4bbe21e1b4ade2ca23acce86a115cb0f59ff.zip | |
ipn/ipn{local,server}: remove ResetForClientDisconnect in favor of SetCurrentUser(nil)
There’s (*LocalBackend).ResetForClientDisconnect, and there’s also (*LocalBackend).resetForProfileChangeLockedOnEntry.
Both methods essentially did the same thing but in slightly different ways. For example, resetForProfileChangeLockedOnEntry didn’t reset the control client until (*LocalBackend).Start() was called at the very end and didn’t reset the keyExpired flag, while ResetForClientDisconnect didn’t reinitialize TKA.
Since SetCurrentUser can be called with a nil argument to reset the currently connected user and internally calls resetForProfileChangeLockedOnEntry, we can remove ResetForClientDisconnect and let SetCurrentUser and resetForProfileChangeLockedOnEntry handle it.
Updates #14823
Signed-off-by: Nick Khyl <nickk@tailscale.com>
Diffstat (limited to 'control/controlhttp/controlhttpserver/controlhttpserver.go')
0 files changed, 0 insertions, 0 deletions