CodeCow/stack
1
0
Fork 0
mirror of https://github.com/stack-auth/stack.git synced 2026-06-13 21:01:21 +08:00
Code Issues Actions 2 Packages Projects Releases Wiki Activity

Log request ID with internal server errors
Some checks failed
Ensure Prisma migrations are in sync with the schema / check_prisma_migrations (22.x) (push) Has been cancelled
Details
Docker Build and Push / Docker Build and Push Server (push) Has been cancelled
Details
Docker Test / docker (push) Has been cancelled
Details
Runs E2E API Tests / build (20.x) (push) Has been cancelled
Details
Runs E2E API Tests / build (22.x) (push) Has been cancelled
Details
Runs E2E API Tests / build (latest) (push) Has been cancelled
Details
Lint & build / lint_and_build (20.x) (push) Has been cancelled
Details
Lint & build / lint_and_build (22.x) (push) Has been cancelled
Details
Dev Environment Test / test (push) Has been cancelled
Details
Run setup tests / test (push) Has been cancelled
Details
TOC Generator / TOC Generator (push) Has been cancelled
Details

Browse Source
This commit is contained in:
Konstantin Wohlwend 2025-02-28 16:56:50 -08:00
parent cbfabe00da
commit 042745a4be
1 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
apps/backend/src/route-handlers/smart-route-handler.tsx
View File

@ -14,10 +14,10 @@ import { DeepPartialSmartRequestWithSentinel, MergeSmartRequest, SmartRequest, c
import { SmartResponse, createResponse, validateSmartResponse } from "./smart-response";
class InternalServerError extends StatusError {
constructor(error: unknown) {
constructor(error: unknown, requestId: string) {
super(
StatusError.InternalServerError,
["development", "test"].includes(getNodeEnvironment()) ? `Internal Server Error. The error message follows, but will be stripped in production. ${errorToNiceString(error)}` : `Something went wrong. Please make sure the data you entered is correct.`,
["development", "test"].includes(getNodeEnvironment()) ? `Internal Server Error. The error message follows, but will be stripped in production. ${errorToNiceString(error)}` : `Something went wrong. Please make sure the data you entered is correct.\n\nRequest ID: ${requestId}`,
);
}
}
@ -36,7 +36,7 @@ const commonErrors = [
* Catches the given error, logs it if needed and returns it as a StatusError. Errors that are not actually errors
* (such as Next.js redirects) will be re-thrown.
*/
function catchError(error: unknown): StatusError {
function catchError(error: unknown, requestId: string): StatusError {
// catch some Next.js non-errors and rethrow them
if (error instanceof Error) {
const digest = (error as any)?.digest;
@ -49,7 +49,7 @@ function catchError(error: unknown): StatusError {
if (error instanceof StatusError) return error;
captureError(`route-handler`, error);
return new InternalServerError(error);
return new InternalServerError(error, requestId);
}
/**
@ -111,7 +111,7 @@ export function handleApiRequest(handler: (req: NextRequest, options: any, reque
} catch (e) {
let statusError: StatusError;
try {
statusError = catchError(e);
statusError = catchError(e, requestId);
} catch (e) {
if (!disableExtendedLogging) console.log(`[ EXC] [${requestId}] ${req.method} ${req.url}: Non-error caught (such as a redirect), will be re-thrown. Digest: ${(e as any)?.digest}`);
throw e;
@ -224,7 +224,7 @@ export function createSmartRouteHandler<
if (reqsErrors.length === 1) {
throw reqsErrors[0];
} else {
const caughtErrors = reqsErrors.map(e => catchError(e));
const caughtErrors = reqsErrors.map(e => catchError(e, requestId));
throw createOverloadsError(caughtErrors);
}
}