updated middleware bug
This commit is contained in:
parent
f2739423b0
commit
e6c30a40d9
@ -1,7 +1,7 @@
|
|||||||
{
|
{
|
||||||
"name": "@debros/node",
|
"name": "@debros/node",
|
||||||
"type": "module",
|
"type": "module",
|
||||||
"version": "0.0.11-alpha",
|
"version": "0.0.12-alpha",
|
||||||
"description": "DeBros Node",
|
"description": "DeBros Node",
|
||||||
"main": "dist/index.js",
|
"main": "dist/index.js",
|
||||||
"bin": "dist/cli.js",
|
"bin": "dist/cli.js",
|
||||||
|
6516
pnpm-lock.yaml
generated
6516
pnpm-lock.yaml
generated
File diff suppressed because it is too large
Load Diff
@ -10,7 +10,6 @@ export const applyMiddleware = (app: express.Application) => {
|
|||||||
app.use(
|
app.use(
|
||||||
cors({
|
cors({
|
||||||
origin: (origin, callback) => {
|
origin: (origin, callback) => {
|
||||||
// Allow requests from your frontend in development
|
|
||||||
const allowedOrigins = ['http://localhost:4001'];
|
const allowedOrigins = ['http://localhost:4001'];
|
||||||
if (!origin || allowedOrigins.includes(origin)) {
|
if (!origin || allowedOrigins.includes(origin)) {
|
||||||
callback(null, true);
|
callback(null, true);
|
||||||
@ -20,7 +19,7 @@ export const applyMiddleware = (app: express.Application) => {
|
|||||||
},
|
},
|
||||||
methods: ['GET', 'POST', 'PUT', 'DELETE', 'OPTIONS'],
|
methods: ['GET', 'POST', 'PUT', 'DELETE', 'OPTIONS'],
|
||||||
allowedHeaders: '*',
|
allowedHeaders: '*',
|
||||||
credentials: true, // Enable credentials
|
credentials: true,
|
||||||
})
|
})
|
||||||
);
|
);
|
||||||
|
|
||||||
@ -31,7 +30,7 @@ export const applyMiddleware = (app: express.Application) => {
|
|||||||
// JSON parser middleware
|
// JSON parser middleware
|
||||||
app.use(express.json());
|
app.use(express.json());
|
||||||
|
|
||||||
// Error handling middleware
|
// Error handling middleware (must come after other middleware/routes)
|
||||||
app.use(errorHandlingMiddleware);
|
app.use(errorHandlingMiddleware);
|
||||||
};
|
};
|
||||||
|
|
||||||
@ -73,12 +72,18 @@ export const loggingMiddleware = (req: express.Request, res: express.Response, n
|
|||||||
/**
|
/**
|
||||||
* Global error handling middleware
|
* Global error handling middleware
|
||||||
*/
|
*/
|
||||||
export const errorHandlingMiddleware = (err: any, req: express.Request, res: express.Response) => {
|
export const errorHandlingMiddleware = (
|
||||||
console.error('Unhandled error:', err);
|
err: any,
|
||||||
res.status(err.status || 500).json({
|
_req: express.Request,
|
||||||
|
res: express.Response,
|
||||||
|
_next: express.NextFunction // Add the 'next' parameter
|
||||||
|
) => {
|
||||||
|
console.error(err.stack);
|
||||||
|
res.status(500).json({
|
||||||
error: {
|
error: {
|
||||||
message: 'An unexpected error occurred',
|
message: 'An unexpected error occurred',
|
||||||
detail: process.env.NODE_ENV === 'development' ? err.message : undefined,
|
detail: process.env.NODE_ENV === 'development' ? err.message : undefined,
|
||||||
},
|
},
|
||||||
});
|
});
|
||||||
|
// Optionally call next(err) if you want to pass the error to another handler
|
||||||
};
|
};
|
||||||
|
Loading…
x
Reference in New Issue
Block a user