Improve logging on tRPC routes (#884)
This commit is contained in:
parent
7f0d8d24a3
commit
3002354c5e
7 changed files with 70 additions and 0 deletions
10
.changeset/thick-chairs-learn.md
Normal file
10
.changeset/thick-chairs-learn.md
Normal file
|
@ -0,0 +1,10 @@
|
||||||
|
---
|
||||||
|
"saleor-app-emails-and-messages": patch
|
||||||
|
"saleor-app-products-feed": patch
|
||||||
|
"saleor-app-invoices": patch
|
||||||
|
"saleor-app-cms-v2": patch
|
||||||
|
"saleor-app-taxes": patch
|
||||||
|
"saleor-app-crm": patch
|
||||||
|
---
|
||||||
|
|
||||||
|
Added error logging for exceptions thrown at tRPC routes.
|
|
@ -1,8 +1,18 @@
|
||||||
import * as trpcNext from "@trpc/server/adapters/next";
|
import * as trpcNext from "@trpc/server/adapters/next";
|
||||||
import { createTrpcContext } from "../../../modules/trpc/trpc-context";
|
import { createTrpcContext } from "../../../modules/trpc/trpc-context";
|
||||||
import { appRouter } from "../../../modules/trpc/trpc-app-router";
|
import { appRouter } from "../../../modules/trpc/trpc-app-router";
|
||||||
|
import { createLogger } from "@saleor/apps-shared";
|
||||||
|
|
||||||
|
const logger = createLogger({ name: "tRPC error" });
|
||||||
|
|
||||||
export default trpcNext.createNextApiHandler({
|
export default trpcNext.createNextApiHandler({
|
||||||
router: appRouter,
|
router: appRouter,
|
||||||
createContext: createTrpcContext,
|
createContext: createTrpcContext,
|
||||||
|
onError: ({ path, error }) => {
|
||||||
|
if (error.code === "INTERNAL_SERVER_ERROR") {
|
||||||
|
logger.error(error, `${path} returned error:`);
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
logger.debug(error, `${path} returned error:`);
|
||||||
|
},
|
||||||
});
|
});
|
||||||
|
|
|
@ -1,8 +1,18 @@
|
||||||
import * as trpcNext from "@trpc/server/adapters/next";
|
import * as trpcNext from "@trpc/server/adapters/next";
|
||||||
import { createTrpcContext } from "../../../modules/trpc/trpc-context";
|
import { createTrpcContext } from "../../../modules/trpc/trpc-context";
|
||||||
import { appRouter } from "../../../modules/trpc/trpc-app-router";
|
import { appRouter } from "../../../modules/trpc/trpc-app-router";
|
||||||
|
import { createLogger } from "@saleor/apps-shared";
|
||||||
|
|
||||||
|
const logger = createLogger({ name: "tRPC error" });
|
||||||
|
|
||||||
export default trpcNext.createNextApiHandler({
|
export default trpcNext.createNextApiHandler({
|
||||||
router: appRouter,
|
router: appRouter,
|
||||||
createContext: createTrpcContext,
|
createContext: createTrpcContext,
|
||||||
|
onError: ({ path, error }) => {
|
||||||
|
if (error.code === "INTERNAL_SERVER_ERROR") {
|
||||||
|
logger.error(error, `${path} returned error:`);
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
logger.debug(error, `${path} returned error:`);
|
||||||
|
},
|
||||||
});
|
});
|
||||||
|
|
|
@ -1,8 +1,18 @@
|
||||||
import * as trpcNext from "@trpc/server/adapters/next";
|
import * as trpcNext from "@trpc/server/adapters/next";
|
||||||
import { createTrpcContext } from "../../../modules/trpc/trpc-context";
|
import { createTrpcContext } from "../../../modules/trpc/trpc-context";
|
||||||
import { appRouter } from "../../../modules/trpc/trpc-app-router";
|
import { appRouter } from "../../../modules/trpc/trpc-app-router";
|
||||||
|
import { createLogger } from "@saleor/apps-shared";
|
||||||
|
|
||||||
|
const logger = createLogger({ name: "tRPC error" });
|
||||||
|
|
||||||
export default trpcNext.createNextApiHandler({
|
export default trpcNext.createNextApiHandler({
|
||||||
router: appRouter,
|
router: appRouter,
|
||||||
createContext: createTrpcContext,
|
createContext: createTrpcContext,
|
||||||
|
onError: ({ path, error }) => {
|
||||||
|
if (error.code === "INTERNAL_SERVER_ERROR") {
|
||||||
|
logger.error(error, `${path} returned error:`);
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
logger.debug(error, `${path} returned error:`);
|
||||||
|
},
|
||||||
});
|
});
|
||||||
|
|
|
@ -1,8 +1,18 @@
|
||||||
import * as trpcNext from "@trpc/server/adapters/next";
|
import * as trpcNext from "@trpc/server/adapters/next";
|
||||||
import { createTrpcContext } from "../../../modules/trpc/trpc-context";
|
import { createTrpcContext } from "../../../modules/trpc/trpc-context";
|
||||||
import { appRouter } from "../../../modules/trpc/trpc-app-router";
|
import { appRouter } from "../../../modules/trpc/trpc-app-router";
|
||||||
|
import { createLogger } from "@saleor/apps-shared";
|
||||||
|
|
||||||
|
const logger = createLogger({ name: "tRPC error" });
|
||||||
|
|
||||||
export default trpcNext.createNextApiHandler({
|
export default trpcNext.createNextApiHandler({
|
||||||
router: appRouter,
|
router: appRouter,
|
||||||
createContext: createTrpcContext,
|
createContext: createTrpcContext,
|
||||||
|
onError: ({ path, error }) => {
|
||||||
|
if (error.code === "INTERNAL_SERVER_ERROR") {
|
||||||
|
logger.error(error, `${path} returned error:`);
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
logger.debug(error, `${path} returned error:`);
|
||||||
|
},
|
||||||
});
|
});
|
||||||
|
|
|
@ -1,8 +1,18 @@
|
||||||
import * as trpcNext from "@trpc/server/adapters/next";
|
import * as trpcNext from "@trpc/server/adapters/next";
|
||||||
import { createTrpcContext } from "../../../modules/trpc/trpc-context";
|
import { createTrpcContext } from "../../../modules/trpc/trpc-context";
|
||||||
import { appRouter } from "../../../modules/trpc/trpc-app-router";
|
import { appRouter } from "../../../modules/trpc/trpc-app-router";
|
||||||
|
import { createLogger } from "@saleor/apps-shared";
|
||||||
|
|
||||||
|
const logger = createLogger({ name: "tRPC error" });
|
||||||
|
|
||||||
export default trpcNext.createNextApiHandler({
|
export default trpcNext.createNextApiHandler({
|
||||||
router: appRouter,
|
router: appRouter,
|
||||||
createContext: createTrpcContext,
|
createContext: createTrpcContext,
|
||||||
|
onError: ({ path, error }) => {
|
||||||
|
if (error.code === "INTERNAL_SERVER_ERROR") {
|
||||||
|
logger.error(error, `${path} returned error:`);
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
logger.debug(error, `${path} returned error:`);
|
||||||
|
},
|
||||||
});
|
});
|
||||||
|
|
|
@ -1,8 +1,18 @@
|
||||||
import * as trpcNext from "@trpc/server/adapters/next";
|
import * as trpcNext from "@trpc/server/adapters/next";
|
||||||
import { appRouter } from "../../../modules/trpc/trpc-app-router";
|
import { appRouter } from "../../../modules/trpc/trpc-app-router";
|
||||||
import { createTrpcContext } from "../../../modules/trpc/trpc-context";
|
import { createTrpcContext } from "../../../modules/trpc/trpc-context";
|
||||||
|
import { createLogger } from "@saleor/apps-shared";
|
||||||
|
|
||||||
|
const logger = createLogger({ name: "tRPC error" });
|
||||||
|
|
||||||
export default trpcNext.createNextApiHandler({
|
export default trpcNext.createNextApiHandler({
|
||||||
router: appRouter,
|
router: appRouter,
|
||||||
createContext: createTrpcContext,
|
createContext: createTrpcContext,
|
||||||
|
onError: ({ path, error }) => {
|
||||||
|
if (error.code === "INTERNAL_SERVER_ERROR") {
|
||||||
|
logger.error(error, `${path} returned error:`);
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
logger.debug(error, `${path} returned error:`);
|
||||||
|
},
|
||||||
});
|
});
|
||||||
|
|
Loading…
Reference in a new issue