bugfixes

ccims · Jul 19, 2024 · f5e7460 · f5e7460
1 parent 2b616e6
commit f5e7460
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 27 deletions.
diff --git a/backend/src/api-login/strategy/strategy-instances.controller.ts b/backend/src/api-login/strategy/strategy-instances.controller.ts
@@ -1,5 +1,4 @@
 import {
-    All,
     Body,
     Controller,
     Delete,

diff --git a/backend/src/strategies/StrategyUsingPassport.ts b/backend/src/strategies/StrategyUsingPassport.ts
@@ -33,8 +33,6 @@ export abstract class StrategyUsingPassport extends Strategy {
         );
     }
 
-    private readonly passportInstances: Map<string, passport.Strategy> = new Map();
-
     abstract createPassportStrategyInstance(strategyInstance: StrategyInstance): passport.Strategy;
 
     protected getAdditionalPassportOptions(
@@ -44,27 +42,14 @@ export abstract class StrategyUsingPassport extends Strategy {
         return {};
     }
 
-    getPassportStrategyInstanceFor(strategyInstance: StrategyInstance): passport.Strategy {
-        if (this.passportInstances.has(strategyInstance.id)) {
-            return this.passportInstances.get(strategyInstance.id);
-        } else {
-            const newInstance = this.createPassportStrategyInstance(strategyInstance);
-            this.logger.debug(
-                `Created new passport strategy for strategy ${this.typeName}, instance: ${strategyInstance.id}`,
-            );
-            this.passportInstances.set(strategyInstance.id, newInstance);
-            return newInstance;
-        }
-    }
-
     public override async performAuth(
         strategyInstance: StrategyInstance,
         state: (AuthStateServerData & OAuthAuthorizeServerState) | undefined,
         req: any,
         res: any,
     ): Promise<PerformAuthResult> {
         return new Promise((resolve, reject) => {
-            const passportStrategy = this.getPassportStrategyInstanceFor(strategyInstance);
+            const passportStrategy = this.createPassportStrategyInstance(strategyInstance)
             const jwtService = this.stateJwtService;
             passport.authenticate(
                 passportStrategy,

diff --git a/frontend/src/components/BaseLayout.vue b/frontend/src/components/BaseLayout.vue
@@ -2,19 +2,15 @@
     <div class="root h-100 w-100 d-flex flex-column">
         <div class="header d-flex align-center my-2">
             <div class="ml-5">
-                <router-link to="/">
-                    <v-btn class="d-flex" variant="text" icon size="small">
-                        <img src="@/assets/logo.svg" width="40" />
-                    </v-btn>
-                </router-link>
+                <v-btn class="d-flex" variant="text" icon size="small">
+                    <img src="@/assets/logo.svg" width="40" />
+                </v-btn>
             </div>
             <slot name="header-title">
                 <div class="d-flex align-center">
-                    <router-link to="/">
-                        <v-btn variant="text" class="px-1" min-width="0" rounded="lger">
-                            <span class="text-h6">Gropius</span>
-                        </v-btn>
-                    </router-link>
+                    <v-btn variant="text" class="px-1" min-width="0" rounded="lger">
+                        <span class="text-h6">Gropius</span>
+                    </v-btn>
                 </div>
             </slot>
             <v-spacer />