🐛 Remove all cookies when signing out thus fixing account switching

Apparently, the Stud.IP API also stores session cookies that have a higher precedence than my OAuth header…
stoeffn · Mar 22, 2018 · 7d74930 · 7d74930
1 parent 6cdf11b
commit 7d74930
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/StudApp/StudKit/Api/Api.swift b/StudApp/StudKit/Api/Api.swift
@@ -31,7 +31,7 @@ class Api<Routes: ApiRoutes> {
 
     // MARK: - Life Cycle
 
-    private let session: URLSession
+    let session: URLSession
 
     /// Base `URL` of all requests this instance issues. Any route paths will be appended to it.
     var baseUrl: URL?

diff --git a/StudApp/StudKit/Models/Users/User.swift b/StudApp/StudKit/Models/Users/User.swift
@@ -81,13 +81,13 @@ public final class User: NSManagedObject, CDCreatable, CDIdentifiable, CDSortabl
 // MARK: - Core Data Operations
 
 extension User {
-    private static var currentUserFromDefaults: User? = {
+    private static var currentUserFromDefaults: User? {
         let storageService = ServiceContainer.default[StorageService.self]
         let coreDataService = ServiceContainer.default[CoreDataService.self]
         let currentUserId = storageService.defaults.string(forKey: UserDefaults.userIdKey)
         guard let user = try? fetch(byId: currentUserId, in: coreDataService.viewContext) else { return nil }
         return user
-    }()
+    }
 
     public internal(set) static var current: User? = currentUserFromDefaults {
         didSet {

diff --git a/StudApp/StudKit/StudIp/StudIpService.swift b/StudApp/StudKit/StudIp/StudIpService.swift
@@ -95,6 +95,7 @@ public class StudIpService {
     func signOut() {
         try? (api.authorizing as? PersistableApiAuthorizing)?.removeCredentials()
 
+        api.session.configuration.httpCookieStorage?.removeCookies(since: .distantPast)
         api.baseUrl = nil
         api.authorizing = nil