Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fixing Z_BUF_ERROR when no content #4701

Merged
merged 5 commits into from May 12, 2022
Merged

Fixing Z_BUF_ERROR when no content #4701

Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
d63b3af
Fix Z_BUF_ERROR when no content
lejahmie May 10, 2022
6cab87b
Refactored code
lejahmie May 11, 2022
4fdb2bd
Fix undefined check
lejahmie May 11, 2022
87d4ed7
Merge branch 'master' into bugfix/z_buf_error-no-content
jasonsaayman May 11, 2022
0735975
Merge branch 'master' into bugfix/z_buf_error-no-content
lejahmie May 11, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
9 changes: 7 additions & 2 deletions lib/adapters/http.js
View file
Open in desktop
Expand Up @@ -236,9 +236,14 @@ module.exports = function httpAdapter(config) {
// return the last request in case of redirects
var lastRequest = res.req || req;

// if decompress disabled we should not decompress
if (config.decompress !== false) {
// if no content, but headers still say that it is encoded,
// remove the header not confuse downstream operations
if (data && data.length === 0 && res.headers['content-encoding']) {
delete res.headers['content-encoding'];
}

// if no content, is HEAD request or decompress disabled we should not decompress
if (res.statusCode !== 204 && lastRequest.method !== 'HEAD' && config.decompress !== false) {
switch (res.headers['content-encoding']) {
/*eslint default-case:0*/
case 'gzip':
Expand Down