Fixes

navikt · Aug 28, 2024 · d7f4e28 · d7f4e28
1 parent fd0d046
commit d7f4e28
Show file tree

Hide file tree

Showing 4 changed files with 19 additions and 12 deletions.
diff --git a/frontend/src/hooks/use-user.ts b/frontend/src/hooks/use-user.ts
@@ -1,8 +1,8 @@
-import { skipToken } from '@reduxjs/toolkit/query';
-import { useGetUserQuery, useGetSessionQuery } from '@app/redux-api/user/api';
+import { SkipToken, skipToken } from '@reduxjs/toolkit/query';
+import { useGetSessionQuery, useGetUserQuery } from '@app/redux-api/user/api';
 
-export const useIsAuthenticated = () => {
-  const { data, ...rest } = useGetSessionQuery(undefined, { refetchOnFocus: true });
+export const useIsAuthenticated = (skip?: SkipToken) => {
+  const { data, ...rest } = useGetSessionQuery(skip, { refetchOnFocus: true });
 
   return { ...rest, data: data?.session.active };
 };

diff --git a/frontend/src/logging/logger.ts b/frontend/src/logging/logger.ts
@@ -74,7 +74,8 @@ class FrontendLogger {
 
   public setSessionEndsAt = (sessionEndsAt: string) => {
     this.sessionEndsAt = sessionEndsAt;
-  }
+  };
 }
 
-export const { apiEvent, appEvent, errorEvent, navigationEvent, sessionEvent, setTokenExpires, setSessionEndsAt } = new FrontendLogger();
+export const { apiEvent, appEvent, errorEvent, navigationEvent, sessionEvent, setTokenExpires, setSessionEndsAt } =
+  new FrontendLogger();
diff --git a/frontend/src/redux-api/user/api.ts b/frontend/src/redux-api/user/api.ts
@@ -42,21 +42,20 @@ export const oauthApi = createApi({
   baseQuery: OAUTH_BASE_QUERY,
   tagTypes: ['session'],
   endpoints: (builder) => ({
-    getSession: builder.query<OAuthSessionData | undefined, void>({
+    getSession: builder.query<OAuthSessionData | null, void>({
       query: () => ({ url: '/session', validateStatus: ({ status, ok }) => ok || status === 401 }),
       providesTags: ['session'],
       onQueryStarted: async (_, { dispatch, queryFulfilled }) => {
         const { data } = await queryFulfilled;
 
-        const hasData = data !== undefined;
-
+        const hasData = data !== null;
 
         if (hasData) {
           setTokenExpires(data.tokens.expire_at);
           setSessionEndsAt(data.session.ends_at);
         }
 
-        if (!hasData || data.session.active !== true) {
+        if (!hasData || !data.session.active) {
           dispatch(userApi.util.updateQueryData('getUser', undefined, () => undefined));
         }
       },

diff --git a/frontend/src/redux/configure-store.ts b/frontend/src/redux/configure-store.ts
@@ -30,7 +30,8 @@ const rtkQueryErrorLogger: Middleware = () => (next) => (action) => {
     console.error('rtkQueryError', action);
 
     if (action.payload.status === 401) {
-      userApi.util.invalidateTags(['isAuthenticated']);
+      userApi.util.invalidateTags(['user']);
+      oauthApi.util.invalidateTags(['session']);
     }
   }
 
@@ -52,7 +53,13 @@ export const reduxStore = configureStore({
           'meta.arg.originalArgs.file',
         ],
       },
-    }).concat([innsendingsytelserApi.middleware, userApi.middleware, caseApi.middleware, oauthApi.middleware, rtkQueryErrorLogger]),
+    }).concat([
+      innsendingsytelserApi.middleware,
+      userApi.middleware,
+      caseApi.middleware,
+      oauthApi.middleware,
+      rtkQueryErrorLogger,
+    ]),
 });
 
 export type AppDispatch = typeof reduxStore.dispatch;