fix: hot fix for syncOwnership

SocialGouv · Feb 9, 2024 · 5f75314 · 5f75314
1 parent 0ad56b7
commit 5f75314
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/packages/app/src/api/core-domain/infra/auth/config.ts b/packages/app/src/api/core-domain/infra/auth/config.ts
@@ -1,6 +1,7 @@
 import { type MonCompteProProfile, MonCompteProProvider } from "@api/core-domain/infra/auth/MonCompteProProvider";
 import { ownershipRepo } from "@api/core-domain/repo";
 import { SyncOwnership } from "@api/core-domain/useCases/SyncOwnership";
+import { logger } from "@api/utils/pino";
 import { config } from "@common/config";
 import { assertImpersonatedSession } from "@common/core-domain/helpers/impersonate";
 import { Octokit } from "@octokit/rest";
@@ -147,8 +148,12 @@ export const authConfig: AuthOptions = {
       } else {
         const sirenList = profile?.organizations.map(orga => orga.siret.substring(0, 9));
         if (profile?.email && sirenList) {
-          const useCase = new SyncOwnership(ownershipRepo);
-          await useCase.execute({ sirens: sirenList, email: profile.email });
+          try {
+            const useCase = new SyncOwnership(ownershipRepo);
+            await useCase.execute({ sirens: sirenList, email: profile.email });
+          } catch (error: unknown) {
+            logger.error("Error while syncing ownerships", error);
+          }
         }
         token.user.companies =
           profile?.organizations.map(orga => ({

diff --git a/packages/app/src/app/(default)/login/page.tsx b/packages/app/src/app/(default)/login/page.tsx
@@ -1,5 +1,4 @@
 import { authConfig } from "@api/core-domain/infra/auth/config";
-import { logger } from "@api/utils/pino";
 import Alert from "@codegouvfr/react-dsfr/Alert";
 import { type NextServerPageProps } from "@common/utils/next";
 import { Box, CenteredContainer } from "@design-system";
@@ -38,10 +37,6 @@ const LoginPage = async ({ searchParams }: NextServerPageProps<never, "callbackU
   const callbackUrl = typeof searchParams.callbackUrl === "string" ? searchParams.callbackUrl : "";
   const error = typeof searchParams.error === "string" ? searchParams.error : "";
 
-  logger.child({ callbackUrl }).info(`LoginPage, callbackUrl:`);
-  logger.child({ error }).info(`LoginPage, error:`);
-  logger.child({ authConfig }).info(`auth config`);
-
   return (
     <CenteredContainer py="6w">
       <h1>{title}</h1>