From 8d8d2b592cf38beaaa06333c72973f3f98db7979 Mon Sep 17 00:00:00 2001 From: Hongbo Wu Date: Tue, 25 Jul 2023 18:18:40 +0800 Subject: [PATCH] move sentry error handler after apollo middleware --- packages/api/src/server.ts | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/packages/api/src/server.ts b/packages/api/src/server.ts index 3a936c60a..fcb009174 100755 --- a/packages/api/src/server.ts +++ b/packages/api/src/server.ts @@ -170,9 +170,6 @@ export const createApp = (): { throw new Error('Sentry TEST error!') }) - // The error handler must be before any other error middleware and after all routes - app.use(Sentry.Handlers.errorHandler()) - const apollo = makeApolloServer() const httpServer = createServer(app) @@ -193,6 +190,9 @@ const main = async (): Promise => { await apollo.start() apollo.applyMiddleware({ app, path: '/api/graphql', cors: corsConfig }) + // The error handler must be before any other error middleware and after all routes + app.use(Sentry.Handlers.errorHandler()) + if (!env.dev.isLocal) { const mwLogger = loggers.get('express', { levels: config.syslog.levels }) const transport = buildLoggerTransport('express')