Merge pull request #400 from btpworks/fix/userinfo-error

do not tolerate failure response from userinfo endpoint
chronicleworks · Jul 25, 2023 · faca7f1 · faca7f1
2 parents 8bd8f92 + faad43a
commit faca7f1
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/crates/api/src/chronicle_graphql/authorization.rs b/crates/api/src/chronicle_graphql/authorization.rs
@@ -144,14 +144,14 @@ impl TokenChecker {
                             ),
                         })
                     }
-                } else if error.is_some() {
-                    tracing::trace!("first error before UserInfo was {error:?}");
+                } else {
+                    if error.is_some() {
+                        tracing::trace!("first error before UserInfo was {error:?}");
+                    }
                     error = Some(Error::UserInfoResponse {
                         server: userinfo_uri.clone(),
                         status: response.status(),
                     });
-                } else {
-                    cache.cache_set(token.to_string(), Map::new());
                 }
             }
         }