fix: just return empty object for empty responses

[etamponi]

This avoids JSON parse errors

[zkochan]

I think it would be correct to return undefined in this case or the raw value of response.body.

So it can be just return response.body && JSON.parse(response.body)

Also, when does it happen? Can a test be added?

[etamponi]

I see it happening when I run `pnpm install --ignore-scripts` the first time. The only dependency I have in the example is `express`. I don't think the method can return undefined, the method signature says PromiseLike<object>. Il 07 gen 2018 02:12, "Zoltan Kochan" <notifications@github.com> ha scritto: I think it would be correct to return undefined in this case or the raw value of response.body. So it can be just return response.body && JSON.parse(response.body) Also, when does it happen? Can a test be added? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#9 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAjUNM3N2tpYq8vSmFGMmRASj6bbBmPYks5tIBoPgaJpZM4RVfoP> .

[zkochan]

You can change the function signature to Promise<object | undefined>

[etamponi]

Ok, it now returns undefined

[zkochan]

ok, if you are able to then add a test. I created an issue #10