Skip to content
This repository was archived by the owner on Sep 29, 2025. It is now read-only.

Commit 393eeeb

Browse files
authored
update CORS error handling (#846)
1 parent 64bcf4a commit 393eeeb

File tree

1 file changed

+13
-12
lines changed
  • packages/mongodb-chatbot-server/src

1 file changed

+13
-12
lines changed

packages/mongodb-chatbot-server/src/app.ts

Lines changed: 13 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,23 +1,22 @@
1+
import "dotenv/config";
12
import express, {
2-
Express,
3-
ErrorRequestHandler,
4-
RequestHandler,
5-
NextFunction,
6-
Request as ExpressRequest,
7-
Response as ExpressResponse,
3+
type Express,
4+
type ErrorRequestHandler,
5+
type RequestHandler,
6+
type NextFunction,
7+
type Request as ExpressRequest,
8+
type Response as ExpressResponse,
89
} from "express";
9-
import cors from "cors";
10-
import "dotenv/config";
10+
import cors, { type CorsOptions } from "cors";
1111
import {
12-
ConversationsRouterParams,
1312
makeConversationsRouter,
14-
ResponsesRouterParams,
1513
makeResponsesRouter,
14+
type ConversationsRouterParams,
15+
type ResponsesRouterParams,
1616
} from "./routes";
1717
import { logger } from "mongodb-rag-core";
1818
import { ObjectId } from "mongodb-rag-core/mongodb";
1919
import { getRequestId, logRequest, sendErrorResponse } from "./utils";
20-
import { CorsOptions } from "cors";
2120
import cloneDeep from "lodash.clonedeep";
2221

2322
/**
@@ -68,7 +67,9 @@ export interface AppConfig {
6867
*/
6968
export const errorHandler: ErrorRequestHandler = (err, req, res, _next) => {
7069
const reqId = getRequestId(req);
71-
const httpStatus = err.status || 500;
70+
const isCorsError = err.message.includes("CORS");
71+
72+
const httpStatus = isCorsError ? 403 : err.status || 500;
7273
const errorMessage = err.message || "Internal Server Error";
7374

7475
if (!res.headersSent) {

0 commit comments

Comments
 (0)