log on token check failure other than 401

ethanndickson · ethanndickson · commit d15b6de58f15 · 2025-03-12T13:29:03.000+11:00
diff --git a/Coder Desktop/Coder Desktop/State.swift b/Coder Desktop/Coder Desktop/State.swift
@@ -2,10 +2,12 @@ import CoderSDK
 import Foundation
 import KeychainAccess
 import NetworkExtension
+import os
 import SwiftUI
 
 @MainActor
 class AppState: ObservableObject {
+    private let logger = Logger(subsystem: Bundle.main.bundleIdentifier!, category: "AppState")
     let appId = Bundle.main.bundleIdentifier!
 
     // Stored in UserDefaults
@@ -96,7 +98,7 @@ class AppState: ObservableObject {
         if hasSession {
             _sessionToken = Published(initialValue: keychainGet(for: Keys.sessionToken))
             if sessionToken == nil || sessionToken!.isEmpty == true {
-                hasSession = false
+                clearSession()
             }
         }
     }
@@ -120,6 +122,7 @@ class AppState: ObservableObject {
                 }
             } catch {
                 // Some other failure, we'll show an error if they try and do something
+                logger.error("failed to check token validity: \(error)")
                 return
             }
         }

Original file line number	Diff line number	Diff line change
`@@ -2,10 +2,12 @@ import CoderSDK`
`2`	`2`	`import Foundation`
`3`	`3`	`import KeychainAccess`
`4`	`4`	`import NetworkExtension`
	`5`	`+import os`
`5`	`6`	`import SwiftUI`
`6`	`7`
`7`	`8`	`@MainActor`
`8`	`9`	`class AppState: ObservableObject {`
	`10`	`+ private let logger = Logger(subsystem: Bundle.main.bundleIdentifier!, category: "AppState")`
`9`	`11`	`let appId = Bundle.main.bundleIdentifier!`
`10`	`12`
`11`	`13`	`// Stored in UserDefaults`
`@@ -96,7 +98,7 @@ class AppState: ObservableObject {`
`96`	`98`	`if hasSession {`
`97`	`99`	`_sessionToken = Published(initialValue: keychainGet(for: Keys.sessionToken))`
`98`	`100`	`if sessionToken == nil \|\| sessionToken!.isEmpty == true {`
`99`		`- hasSession = false`
	`101`	`+ clearSession()`
`100`	`102`	`}`
`101`	`103`	`}`
`102`	`104`	`}`
`@@ -120,6 +122,7 @@ class AppState: ObservableObject {`
`120`	`122`	`}`
`121`	`123`	`} catch {`
`122`	`124`	`// Some other failure, we'll show an error if they try and do something`
	`125`	`+ logger.error("failed to check token validity: \(error)")`
`123`	`126`	`return`
`124`	`127`	`}`
`125`	`128`	`}`