fix(proxy): place keys in correct place (#2209)
This commit is contained in:
@@ -21,7 +21,13 @@ const settings: ServerSetting = {
|
|||||||
'sec-fetch-dest',
|
'sec-fetch-dest',
|
||||||
'pragma',
|
'pragma',
|
||||||
],
|
],
|
||||||
disAllowResponseHeaders: ['link', 'set-cookie', 'set-cookie2'],
|
disAllowResponseHeaders: [
|
||||||
|
'link',
|
||||||
|
'set-cookie',
|
||||||
|
'set-cookie2',
|
||||||
|
'content-encoding',
|
||||||
|
'content-length',
|
||||||
|
],
|
||||||
useUserAgent: true,
|
useUserAgent: true,
|
||||||
};
|
};
|
||||||
|
|
||||||
@@ -160,11 +166,7 @@ const proxyRequest: Connect.SimpleHandleFunction = (req, res) => {
|
|||||||
.then(async res2 => {
|
.then(async res2 => {
|
||||||
res.statusCode = res2.status;
|
res.statusCode = res2.status;
|
||||||
res2.headers.forEach((val, key) => {
|
res2.headers.forEach((val, key) => {
|
||||||
if (
|
if (!settings.disAllowResponseHeaders.includes(key)) {
|
||||||
!settings.disAllowResponseHeaders.includes(key) &&
|
|
||||||
key !== 'content-encoding' &&
|
|
||||||
key !== 'content-length'
|
|
||||||
) {
|
|
||||||
res.setHeader(key, val);
|
res.setHeader(key, val);
|
||||||
}
|
}
|
||||||
});
|
});
|
||||||
@@ -234,11 +236,7 @@ proxy.on('proxyRes', function (proxyRes, req, res) {
|
|||||||
|
|
||||||
// Propagate headers but filter restricted ones
|
// Propagate headers but filter restricted ones
|
||||||
Object.keys(proxyRes.headers).forEach(key => {
|
Object.keys(proxyRes.headers).forEach(key => {
|
||||||
if (
|
if (!settings.disAllowResponseHeaders.includes(key)) {
|
||||||
!settings.disAllowResponseHeaders.includes(key) &&
|
|
||||||
key !== 'content-encoding' &&
|
|
||||||
key !== 'content-length'
|
|
||||||
) {
|
|
||||||
res.setHeader(key, proxyRes.headers[key] as string);
|
res.setHeader(key, proxyRes.headers[key] as string);
|
||||||
}
|
}
|
||||||
});
|
});
|
||||||
|
|||||||
Reference in New Issue
Block a user