Skip to content

Commit

Permalink
fix: filter proxy request headers
Browse files Browse the repository at this point in the history
Remove sensitive headers in separate middleware
  • Loading branch information
cgawron committed Apr 6, 2024
1 parent 21d17a9 commit 8eb9579
Showing 1 changed file with 12 additions and 8 deletions.
20 changes: 12 additions & 8 deletions src/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -206,6 +206,18 @@ app.use((req, res, next) => {

});

/** redact headers */
app.use((req, res, next) => {
const redactHeaders = Object.keys(req.headers)
.filter((header) => header == 'cookie' || header.startsWith('x-'))
logger.debug('redactHeaders', redactHeaders);
redactHeaders.forEach((header) => {
req.headers[header] = 'redacted';
delete req.headers[header];
})
next();
});

app.get('/user', (req, res) => {
res.send(req['user']);
});
Expand Down Expand Up @@ -273,14 +285,6 @@ app.use(`${PREFIX}*`,
proxyReqOpts.headers['authorization'] = `Bearer ${API_KEY}`;
proxyReqOpts.headers['OpenAI-Beta'] = 'assistants=v1';
logger.debug('srcReq.headers', srcReq.headers);
const redactHeaders = Object.keys(proxyReqOpts.headers)
.filter((header) => header == 'cookie' || header.startsWith('x-'))
logger.debug('redactHeaders', redactHeaders);
redactHeaders.forEach((header) => {
proxyReqOpts.headers[header] = "";
//delete proxyReqOpts.headers[header];
})

logger.debug('proxy headers', proxyReqOpts.headers);
logger.debug('body', srcReq.body, proxyReqOpts.body);
return proxyReqOpts;
Expand Down

0 comments on commit 8eb9579

Please sign in to comment.