Force refreshing token when changing AI chat permissions (fixes #766) (#837)

src/GrampsJs.js

@@ -529,7 +529,7 @@ export class GrampsJs extends LitElement {
       this._handleVisibilityChange()
     )
     window.addEventListener('online', () => this._handleOnline())
-    window.addEventListener('token:refresh', () => this._handleRefresh())
+    window.addEventListener('token:refresh', () => this._handleRefresh(true))
 
     if (window.location.pathname.includes('/oidc/complete')) {
       return
@@ -822,8 +822,8 @@ export class GrampsJs extends LitElement {
   }
 
   // eslint-disable-next-line class-methods-use-this
-  async _handleRefresh() {
-    await this.appState.refreshTokenIfNeeded()
+  async _handleRefresh(force = false) {
+    await this.appState.refreshTokenIfNeeded(force)
     this.setPermissions()
   }
 

src/api.js

@@ -678,8 +678,8 @@ export class Auth {
     return localStorage.getItem('refresh_token')
   }
 
-  async getValidAccessToken() {
-    if (this._shouldRefresh()) {
+  async getValidAccessToken(force = false) {
+    if (force || this._shouldRefresh()) {
       if (this._refreshingTokens) {
         // If already refreshing, wait for that to finish
         await this._refreshingTokens

src/appState.js

@@ -41,7 +41,7 @@ export function getInitialAppState() {
       apiPutPostDeleteNew(auth, 'PUT', endpoint, payload, options),
     apiDelete: (endpoint, options = {}) =>
       apiPutPostDeleteNew(auth, 'DELETE', endpoint, {}, options),
-    refreshTokenIfNeeded: () => auth.getValidAccessToken(),
+    refreshTokenIfNeeded: (force = false) => auth.getValidAccessToken(force),
     signout: () => auth.signout(),
     updateSettings: (settings = {}, tree = false) =>
       updateSettings(settings, tree),