fix: filter out internal content-type headers for no body requests in axiosinstance (#5591)

* fix: filter out internal content-type headers for no body requests in axios instance
This commit is contained in:
Sanjai Kumar
2025-10-08 17:25:21 +05:30
committed by GitHub
parent c6ce40c245
commit ce40949564
2 changed files with 26 additions and 2 deletions

View File

@@ -126,7 +126,13 @@ function makeAxiosInstance({
type: 'request',
message: `${config.method.toUpperCase()} ${config.url}`,
});
Object.entries(config.headers).forEach(([key, value]) => {
// Skip Bruno's internal content-type: false for no body requests
if (key.toLowerCase() === 'content-type' && value === false) {
return;
}
timeline.push({
timestamp: new Date(),
type: 'requestHeader',

View File

@@ -613,10 +613,19 @@ const registerNetworkIpc = (mainWindow) => {
);
const { data: requestData, dataBuffer: requestDataBuffer } = parseDataFromRequest(request);
// Remove false Content-Type header (used to stop axios from auto-setting it); no Content-Type was actually set or sent.
const headersSent = { ...request.headers };
Object.keys(headersSent).forEach((key) => {
if (key.toLowerCase() === 'content-type' && headersSent[key] === false) {
delete headersSent[key];
}
});
let requestSent = {
url: request.url,
method: request.method,
headers: request.headers,
headers: headersSent,
data: requestData,
dataBuffer: requestDataBuffer
}
@@ -1090,10 +1099,19 @@ const registerNetworkIpc = (mainWindow) => {
}
const { data: requestData, dataBuffer: requestDataBuffer } = parseDataFromRequest(request);
// Remove false Content-Type header (used to stop axios from auto-setting it); no Content-Type was actually set or sent.
const headersSent = { ...request.headers };
Object.keys(headersSent).forEach((key) => {
if (key.toLowerCase() === 'content-type' && headersSent[key] === false) {
delete headersSent[key];
}
});
let requestSent = {
url: request.url,
method: request.method,
headers: request.headers,
headers: headersSent,
data: requestData,
dataBuffer: requestDataBuffer
}