Skip to content

Commit 96121d5

Browse files
author
Nagarakanti Kiran
authored
Update requestLogger.js
Before: Using require() for modern modules that don't support it After: Switched to dynamic import() which works with all module types
1 parent 1f0fcd1 commit 96121d5

1 file changed

Lines changed: 13 additions & 4 deletions

File tree

server/middleware/requestLogger.js

Lines changed: 13 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -29,10 +29,19 @@ const errorLogStream = fs.createWriteStream(
2929
);
3030

3131
// Middleware to add request ID
32-
const addRequestId = (req, res, next) => {
33-
req.reqId = require('nanoid').nanoid(8);
34-
res.setHeader('X-Request-ID', req.reqId);
35-
next();
32+
const addRequestId = async (req, res, next) => {
33+
try {
34+
const { nanoid } = await import('nanoid');
35+
req.reqId = nanoid(8);
36+
res.setHeader('X-Request-ID', req.reqId);
37+
next();
38+
} catch (error) {
39+
console.error('Error generating request ID:', error);
40+
// Fallback ID
41+
req.reqId = Math.random().toString(36).substr(2, 8);
42+
res.setHeader('X-Request-ID', req.reqId);
43+
next();
44+
}
3645
};
3746

3847
// Standard request logger

0 commit comments

Comments
 (0)