From 39f2a03bacd2719fae26a6220d26c8963b324b19 Mon Sep 17 00:00:00 2001
From: Bastien Dumont <bdumont@grandlyon.com>
Date: Mon, 25 Sep 2023 15:16:48 +0200
Subject: [PATCH] capture error
---
.../EPGLConnect/FormLogin/FormLogin.tsx | 4 ++-
src/components/Hooks/useKonnectorAuth.tsx | 2 +-
src/components/Splash/SplashRoot.tsx | 2 +-
src/migrations/migration.ts | 2 +-
src/services/account.service.ts | 10 +++---
src/services/challenge.service.ts | 8 ++---
src/services/customPopup.service.ts | 2 +-
src/services/duel.service.ts | 2 +-
src/services/ecogesture.service.ts | 8 ++---
.../enedisMonthlyAnalysisData.service.ts | 2 +-
src/services/fluidsPrices.service.ts | 4 +--
src/services/initialization.service.ts | 34 +++++++++----------
src/services/mail.service.ts | 2 +-
src/services/partnersInfo.service.ts | 2 +-
src/services/profileType.service.ts | 2 +-
src/services/profileTypeEntity.service.ts | 2 +-
src/services/terms.service.ts | 2 +-
src/services/triggers.service.ts | 2 +-
src/services/usageEvent.service.ts | 4 +--
src/targets/services/aggregatorUsageEvents.ts | 4 +--
src/targets/services/fluidsPrices.ts | 6 ++--
.../services/monthlyReportNotification.ts | 4 +--
22 files changed, 56 insertions(+), 54 deletions(-)
diff --git a/src/components/Connection/EPGLConnect/FormLogin/FormLogin.tsx b/src/components/Connection/EPGLConnect/FormLogin/FormLogin.tsx
index 118d594a9..f2216ea1d 100644
--- a/src/components/Connection/EPGLConnect/FormLogin/FormLogin.tsx
+++ b/src/components/Connection/EPGLConnect/FormLogin/FormLogin.tsx
@@ -9,6 +9,7 @@ import { FluidType } from 'enums'
import { Account, AccountAuthData } from 'models'
import React, { useEffect, useState } from 'react'
import { useAppSelector } from 'store/hooks'
+import logApp from 'utils/logger'
import { getPartnerPicto } from 'utils/picto'
import './formLogin.scss'
@@ -75,7 +76,8 @@ const FormLogin = () => {
await update()
}
} catch (error) {
- Sentry.captureException(JSON.stringify({ error }))
+ logApp.error(error)
+ Sentry.captureException(error)
setLoading(false)
}
}
diff --git a/src/components/Hooks/useKonnectorAuth.tsx b/src/components/Hooks/useKonnectorAuth.tsx
index e0f926074..1750eff2d 100644
--- a/src/components/Hooks/useKonnectorAuth.tsx
+++ b/src/components/Hooks/useKonnectorAuth.tsx
@@ -83,7 +83,7 @@ const useKonnectorAuth = (
setLoading(false)
sendUsageEventError(konnectorSlug)
logApp.error(error)
- Sentry.captureException(JSON.stringify({ error }))
+ Sentry.captureException(error)
}
}
diff --git a/src/components/Splash/SplashRoot.tsx b/src/components/Splash/SplashRoot.tsx
index a0cfaf2a2..49d280d31 100644
--- a/src/components/Splash/SplashRoot.tsx
+++ b/src/components/Splash/SplashRoot.tsx
@@ -373,7 +373,7 @@ const SplashRoot = ({ fadeTimer = 1000, children }: SplashRootProps) => {
setInitStepErrors(InitStepsErrors.UNKNOWN_ERROR)
}
logApp.error(`[Initialization] Error : ${error}`)
- Sentry.captureException(JSON.stringify({ error }))
+ Sentry.captureException(error)
} finally {
transaction.finish()
}
diff --git a/src/migrations/migration.ts b/src/migrations/migration.ts
index b5e6819cb..22c34e25b 100644
--- a/src/migrations/migration.ts
+++ b/src/migrations/migration.ts
@@ -181,7 +181,7 @@ export async function migrate(
}
} catch (error: any) {
console.error(error)
- Sentry.captureException(JSON.stringify({ error }))
+ Sentry.captureException(error)
result = {
type: MIGRATION_RESULT_FAILED,
errors: [error.toString()],
diff --git a/src/services/account.service.ts b/src/services/account.service.ts
index 13ffc78f8..3ed4c8bfc 100644
--- a/src/services/account.service.ts
+++ b/src/services/account.service.ts
@@ -57,7 +57,7 @@ export default class AccountService {
const errorMessage = `Get account failed :${JSON.stringify(error)}`
logStack('error', errorMessage)
logApp.error(errorMessage)
- Sentry.captureException(errorMessage)
+ Sentry.captureException(error)
throw new Error('Get account failed')
}
}
@@ -106,7 +106,7 @@ export default class AccountService {
const errorMessage = `Error GetAccountByType: ${JSON.stringify(error)}`
logStack('error', errorMessage)
logApp.error(errorMessage)
- Sentry.captureException(errorMessage)
+ Sentry.captureException(error)
return null
}
}
@@ -124,7 +124,7 @@ export default class AccountService {
const errorMessage = `Error: GetAccountsByType: ${JSON.stringify(error)}`
logStack('error', errorMessage)
logApp.error(errorMessage)
- Sentry.captureException(errorMessage)
+ Sentry.captureException(error)
return []
}
}
@@ -137,7 +137,7 @@ export default class AccountService {
const errorMessage = `Update account failed: ${JSON.stringify(error)}`
logStack('error', errorMessage)
logApp.error(errorMessage)
- Sentry.captureException(errorMessage)
+ Sentry.captureException(error)
throw new Error('Update account failed')
}
}
@@ -150,7 +150,7 @@ export default class AccountService {
const errorMessage = `Delete account failed`
logStack('error', errorMessage)
logApp.error(errorMessage)
- Sentry.captureException(errorMessage)
+ Sentry.captureException(error)
throw new Error(errorMessage)
}
}
diff --git a/src/services/challenge.service.ts b/src/services/challenge.service.ts
index ff4623e38..7a1ad1697 100644
--- a/src/services/challenge.service.ts
+++ b/src/services/challenge.service.ts
@@ -485,7 +485,7 @@ export default class ChallengeService {
)}`
logStack('error', errorMessage)
logApp.error(errorMessage)
- Sentry.captureException(errorMessage)
+ Sentry.captureException(error)
throw error
}
}
@@ -540,7 +540,7 @@ export default class ChallengeService {
)}`
logStack('error', errorMessage)
logApp.error(errorMessage)
- Sentry.captureException(errorMessage)
+ Sentry.captureException(error)
throw error
}
}
@@ -583,7 +583,7 @@ export default class ChallengeService {
)}`
logStack('error', errorMessage)
logApp.error(errorMessage)
- Sentry.captureException(errorMessage)
+ Sentry.captureException(error)
throw error
}
}
@@ -803,7 +803,7 @@ export default class ChallengeService {
)}`
logStack('error', errorMessage)
logApp.error(errorMessage)
- Sentry.captureException(errorMessage)
+ Sentry.captureException(error)
throw error
}
}
diff --git a/src/services/customPopup.service.ts b/src/services/customPopup.service.ts
index 71c308900..f09472520 100644
--- a/src/services/customPopup.service.ts
+++ b/src/services/customPopup.service.ts
@@ -39,7 +39,7 @@ export default class CustomPopupService {
)}`
logStack('error', errorMessage)
logApp.error(errorMessage)
- Sentry.captureException(errorMessage)
+ Sentry.captureException(error)
}
}
}
diff --git a/src/services/duel.service.ts b/src/services/duel.service.ts
index 204e9e708..e18c62a54 100644
--- a/src/services/duel.service.ts
+++ b/src/services/duel.service.ts
@@ -156,7 +156,7 @@ export default class DuelService {
const errorMessage = `deleteAllDuelEntities:${JSON.stringify(error)}`
logStack('error', errorMessage)
logApp.error(errorMessage)
- Sentry.captureException(errorMessage)
+ Sentry.captureException(error)
throw error
}
}
diff --git a/src/services/ecogesture.service.ts b/src/services/ecogesture.service.ts
index 402795d9e..ce32ccb52 100644
--- a/src/services/ecogesture.service.ts
+++ b/src/services/ecogesture.service.ts
@@ -62,7 +62,7 @@ export default class EcogestureService {
)}`
logStack('error', errorMessage)
logApp.error(errorMessage)
- Sentry.captureException(errorMessage)
+ Sentry.captureException(error)
throw error
}
}
@@ -102,7 +102,7 @@ export default class EcogestureService {
)}`
logStack('error', errorMessage)
logApp.error(errorMessage)
- Sentry.captureException(errorMessage)
+ Sentry.captureException(error)
throw error
}
} else {
@@ -178,7 +178,7 @@ export default class EcogestureService {
)}`
logStack('error', errorMessage)
logApp.error(errorMessage)
- Sentry.captureException(errorMessage)
+ Sentry.captureException(error)
throw error
}
}
@@ -201,7 +201,7 @@ export default class EcogestureService {
)}`
logStack('error', errorMessage)
logApp.error(errorMessage)
- Sentry.captureException(errorMessage)
+ Sentry.captureException(error)
throw error
}
}
diff --git a/src/services/enedisMonthlyAnalysisData.service.ts b/src/services/enedisMonthlyAnalysisData.service.ts
index 7d893b96e..0b85f2cdc 100644
--- a/src/services/enedisMonthlyAnalysisData.service.ts
+++ b/src/services/enedisMonthlyAnalysisData.service.ts
@@ -145,7 +145,7 @@ export default class EnedisMonthlyAnalysisDataService {
)}`
logStack('error', errorMessage)
logApp.error(errorMessage)
- Sentry.captureException(errorMessage)
+ Sentry.captureException(error)
throw error
}
}
diff --git a/src/services/fluidsPrices.service.ts b/src/services/fluidsPrices.service.ts
index 64ad119f7..9225a454c 100644
--- a/src/services/fluidsPrices.service.ts
+++ b/src/services/fluidsPrices.service.ts
@@ -113,7 +113,7 @@ export default class FluidPricesService {
const errorMessage = `deleteAllFluidsPrices: ${JSON.stringify(error)}`
logStack('error', errorMessage)
logApp.error(errorMessage)
- Sentry.captureException(errorMessage)
+ Sentry.captureException(error)
return false
}
}
@@ -153,7 +153,7 @@ export default class FluidPricesService {
)}`
logStack('error', errorMessage)
logApp.error(errorMessage)
- Sentry.captureException(errorMessage)
+ Sentry.captureException(error)
throw error
}
}
diff --git a/src/services/initialization.service.ts b/src/services/initialization.service.ts
index 8dd6d371c..1383331b4 100644
--- a/src/services/initialization.service.ts
+++ b/src/services/initialization.service.ts
@@ -109,7 +109,7 @@ export default class InitializationService {
)}`
logStack('error', errorMessage)
logApp.error(errorMessage)
- Sentry.captureException(errorMessage)
+ Sentry.captureException(error)
throw error
}
}
@@ -134,7 +134,7 @@ export default class InitializationService {
)}`
logStack('error', errorMessage)
logApp.error(errorMessage)
- Sentry.captureException(errorMessage)
+ Sentry.captureException(error)
throw error
}
}
@@ -153,7 +153,7 @@ export default class InitializationService {
)}`
logStack('error', errorMessage)
logApp.error(errorMessage)
- Sentry.captureException(errorMessage)
+ Sentry.captureException(error)
throw error
}
}
@@ -214,7 +214,7 @@ export default class InitializationService {
)}`
logStack('error', errorMessage)
logApp.error(errorMessage)
- Sentry.captureException(errorMessage)
+ Sentry.captureException(error)
throw error
}
}
@@ -245,7 +245,7 @@ export default class InitializationService {
)}`
logStack('error', errorMessage)
logApp.error(errorMessage)
- Sentry.captureException(errorMessage)
+ Sentry.captureException(error)
throw error
}
} else {
@@ -284,7 +284,7 @@ export default class InitializationService {
)}`
logStack('error', errorMessage)
logApp.error(errorMessage)
- Sentry.captureException(errorMessage)
+ Sentry.captureException(error)
throw error
}
}
@@ -315,7 +315,7 @@ export default class InitializationService {
)}`
logStack('error', errorMessage)
logApp.error(errorMessage)
- Sentry.captureException(errorMessage)
+ Sentry.captureException(error)
throw error
}
} else {
@@ -355,7 +355,7 @@ export default class InitializationService {
)}`
logStack('error', errorMessage)
logApp.error(errorMessage)
- Sentry.captureException(errorMessage)
+ Sentry.captureException(error)
throw error
}
}
@@ -386,7 +386,7 @@ export default class InitializationService {
)}`
logStack('error', errorMessage)
logApp.error(errorMessage)
- Sentry.captureException(errorMessage)
+ Sentry.captureException(error)
throw error
}
} else {
@@ -429,7 +429,7 @@ export default class InitializationService {
)}`
logStack('error', errorMessage)
logApp.error(errorMessage)
- Sentry.captureException(errorMessage)
+ Sentry.captureException(error)
throw error
}
}
@@ -463,7 +463,7 @@ export default class InitializationService {
)}`
logStack('error', errorMessage)
logApp.error(errorMessage)
- Sentry.captureException(errorMessage)
+ Sentry.captureException(error)
throw error
}
} else {
@@ -509,7 +509,7 @@ export default class InitializationService {
)}`
logStack('error', errorMessage)
logApp.error(errorMessage)
- Sentry.captureException(errorMessage)
+ Sentry.captureException(error)
throw error
}
}
@@ -539,7 +539,7 @@ export default class InitializationService {
)}`
logStack('error', errorMessage)
logApp.error(errorMessage)
- Sentry.captureException(errorMessage)
+ Sentry.captureException(error)
throw error
}
}
@@ -569,7 +569,7 @@ export default class InitializationService {
)}`
logStack('error', errorMessage)
logApp.error(errorMessage)
- Sentry.captureException(errorMessage)
+ Sentry.captureException(error)
throw error
}
}
@@ -602,7 +602,7 @@ export default class InitializationService {
)}`
logStack('error', errorMessage)
logApp.error(errorMessage)
- Sentry.captureException(errorMessage)
+ Sentry.captureException(error)
throw error
}
}
@@ -628,7 +628,7 @@ export default class InitializationService {
const errorMessage = `Initialization error - : ${JSON.stringify(error)}`
logStack('error', errorMessage)
logApp.error(errorMessage)
- Sentry.captureException(errorMessage)
+ Sentry.captureException(error)
throw error
}
}
@@ -673,7 +673,7 @@ export default class InitializationService {
)}`
logStack('error', errorMessage)
logApp.error(errorMessage)
- Sentry.captureException(errorMessage)
+ Sentry.captureException(error)
throw error
} finally {
logDuration('[Initialization] initConsent finished', startTime)
diff --git a/src/services/mail.service.ts b/src/services/mail.service.ts
index 9528efdfd..1e425524b 100644
--- a/src/services/mail.service.ts
+++ b/src/services/mail.service.ts
@@ -16,7 +16,7 @@ export default class MailService {
const errorMessage = `Failed to send mail`
logStack('error', errorMessage)
logApp.error(errorMessage)
- Sentry.captureException(errorMessage)
+ Sentry.captureException(error)
throw new Error(errorMessage)
}
}
diff --git a/src/services/partnersInfo.service.ts b/src/services/partnersInfo.service.ts
index 18bf02ce5..624137ddc 100644
--- a/src/services/partnersInfo.service.ts
+++ b/src/services/partnersInfo.service.ts
@@ -39,7 +39,7 @@ export default class PartnersInfoService {
)}`
logStack('error', errorMessage)
logApp.error(errorMessage)
- Sentry.captureException(errorMessage)
+ Sentry.captureException(error)
}
}
}
diff --git a/src/services/profileType.service.ts b/src/services/profileType.service.ts
index a5a2900fc..689a15fdf 100644
--- a/src/services/profileType.service.ts
+++ b/src/services/profileType.service.ts
@@ -533,7 +533,7 @@ export default class ProfileTypeService {
const errorMessage = `fetchDju error : ${JSON.stringify(error)}`
logStack('error', errorMessage)
logApp.error(errorMessage)
- Sentry.captureException(errorMessage)
+ Sentry.captureException(error)
return heatingData.dju_average_by_month[month - 1]
}
}
diff --git a/src/services/profileTypeEntity.service.ts b/src/services/profileTypeEntity.service.ts
index bb68839a1..7e44d4dfa 100644
--- a/src/services/profileTypeEntity.service.ts
+++ b/src/services/profileTypeEntity.service.ts
@@ -168,7 +168,7 @@ export default class ProfileTypeEntityService {
const errorMessage = `deleteProfileTypes: ${JSON.stringify(error)}`
logStack('error', errorMessage)
logApp.error(errorMessage)
- Sentry.captureException(errorMessage)
+ Sentry.captureException(error)
return false
}
}
diff --git a/src/services/terms.service.ts b/src/services/terms.service.ts
index c9c3061a9..ed20aea33 100644
--- a/src/services/terms.service.ts
+++ b/src/services/terms.service.ts
@@ -92,7 +92,7 @@ export default class TermsService {
const errorMessage = `Error creating new term: ${JSON.stringify(error)}`
logStack('error', errorMessage)
logApp.error(errorMessage)
- Sentry.captureException(errorMessage)
+ Sentry.captureException(error)
throw error
}
}
diff --git a/src/services/triggers.service.ts b/src/services/triggers.service.ts
index 830b22746..d3478fcfc 100644
--- a/src/services/triggers.service.ts
+++ b/src/services/triggers.service.ts
@@ -103,7 +103,7 @@ export default class TriggerService {
const errorMessage = `Delete trigger failed: ${JSON.stringify(error)}`
logStack('error', errorMessage)
logApp.error(errorMessage)
- Sentry.captureException(errorMessage)
+ Sentry.captureException(error)
throw new Error('Delete trigger failed')
}
}
diff --git a/src/services/usageEvent.service.ts b/src/services/usageEvent.service.ts
index 7b396168c..fca68e79d 100644
--- a/src/services/usageEvent.service.ts
+++ b/src/services/usageEvent.service.ts
@@ -106,7 +106,7 @@ export default class UsageEventService {
)}`
logStack('error', errorMessage)
logApp.error(errorMessage)
- Sentry.captureException(errorMessage)
+ Sentry.captureException(error)
}
}
@@ -134,7 +134,7 @@ export default class UsageEventService {
)}`
logStack('error', errorMessage)
logApp.error(errorMessage)
- Sentry.captureException(errorMessage)
+ Sentry.captureException(error)
throw new Error('Could not update all events')
}
}
diff --git a/src/targets/services/aggregatorUsageEvents.ts b/src/targets/services/aggregatorUsageEvents.ts
index 6bf0bda5d..56f17c02d 100644
--- a/src/targets/services/aggregatorUsageEvents.ts
+++ b/src/targets/services/aggregatorUsageEvents.ts
@@ -85,7 +85,7 @@ const sendIndicator = async (
'error',
`Error while sending indicator ${indicator.measureName} to remote doctype: ${errorMessage}`
)
- Sentry.captureException(JSON.stringify({ error }))
+ Sentry.captureException(error)
throw error
}
}
@@ -1231,7 +1231,7 @@ const AggregatorUsageEvents = async ({
error += `${el.doctype}, `
})
logStack('error', error)
- Sentry.captureException(JSON.stringify({ error }))
+ Sentry.captureException(error)
throw error
}
diff --git a/src/targets/services/fluidsPrices.ts b/src/targets/services/fluidsPrices.ts
index 8bdfe52b4..21b4e081a 100644
--- a/src/targets/services/fluidsPrices.ts
+++ b/src/targets/services/fluidsPrices.ts
@@ -99,7 +99,7 @@ const synchroPricesToUpdate = async (
})
} catch (error) {
logStack('error', `Error: ${error}`)
- Sentry.captureException(JSON.stringify({ error }))
+ Sentry.captureException(error)
}
})
)
@@ -186,7 +186,7 @@ const aggregatePrices = async (
} while (date < today)
} catch (error) {
logStack('info', `Error : ${error}`)
- Sentry.captureException(JSON.stringify({ error }))
+ Sentry.captureException(error)
}
})
)
@@ -326,7 +326,7 @@ const applyPrices = async (client: Client, fluidType: FluidType) => {
} while (date < today)
} catch (error) {
logStack('error', `ERROR : ${error} `)
- Sentry.captureException(JSON.stringify({ error }))
+ Sentry.captureException(error)
}
})
)
diff --git a/src/targets/services/monthlyReportNotification.ts b/src/targets/services/monthlyReportNotification.ts
index 352409998..5f201313c 100644
--- a/src/targets/services/monthlyReportNotification.ts
+++ b/src/targets/services/monthlyReportNotification.ts
@@ -139,7 +139,7 @@ const getMonthlyReport = async (
return result
} catch (error) {
logStack('error', JSON.stringify(error))
- Sentry.captureException(JSON.stringify({ error }))
+ Sentry.captureException(error)
return {
year: parseInt(year),
month: parseInt(month),
@@ -178,7 +178,7 @@ const monthlyReportNotification = async ({
})
} catch (error) {
logStack('error', 'Update mailToken user profile error : ' + error)
- Sentry.captureException(JSON.stringify({ error }))
+ Sentry.captureException(error)
throw error
}
}
--
GitLab