Cleanup fixes (#53)

* Make idToken a property of User (instead of function). * Remove Client.cachedUser completely.
schibsted · Aug 31, 2022 · f5931b0 · f5931b0
1 parent 9e1d02b
commit f5931b0
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 14 deletions.
diff --git a/webflows/src/main/java/com/schibsted/account/webflows/client/Client.kt b/webflows/src/main/java/com/schibsted/account/webflows/client/Client.kt
@@ -211,24 +211,16 @@ class Client : ClientInterface {
         }
     }
 
-    private var cashedUser: User? = null
-
     /** Resume any previously logged-in user session */
     override fun resumeLastLoggedInUser(callback: (Either<StorageError, User?>) -> Unit) {
         sessionStorage.get(configuration.clientId) { result ->
             result
                 .foreach { storedUserSession: StoredUserSession? ->
                     if (storedUserSession == null) {
-                        cashedUser = null
                         callback(Right(null))
                     } else {
                         val user = User(this, storedUserSession.userTokens)
-                        if (user.equals(cashedUser)) {
-                            callback(Right(cashedUser))
-                        } else {
-                            cashedUser = user
-                            callback(Right(cashedUser))
-                        }
+                        callback(Right(user))
                     }
                 }
                 .left().foreach {

diff --git a/webflows/src/main/java/com/schibsted/account/webflows/user/User.kt b/webflows/src/main/java/com/schibsted/account/webflows/user/User.kt
@@ -47,6 +47,14 @@ class User {
             tokens.idTokenClaims.sub
         }
 
+    /**
+     * ID Token
+     */
+    val idToken: String
+        get() = onlyIfLoggedIn { tokens ->
+            tokens.idToken
+        }
+
     constructor(client: Client, session: UserSession) : this(client, session.tokens)
 
     internal constructor(client: Client, tokens: UserTokens) {
@@ -86,11 +94,6 @@ class User {
         client.schibstedAccountApi.userProfile(this, callback)
     }
 
-    /**
-     * Get idToken for User
-     * */
-    fun getIdToken(): String? = tokens?.idToken
-
     /**
      * Generate URL with embedded one-time code for creating a web session for the current user.
      *