read body to avoid memory leaks

[ykzts]

undici fetch and node-fetch maintain the connection until the body is read, causing a memory leak if the body is not read¹².

In this Pull Request, if a 304 is returned or an error is returned, a meaningless res.arrayBuffer (slow because res.json and res.text are parsed together) is executed to read the body.

Footnotes

1. https://github.com/nodejs/undici/blob/v5.21.2/README.md#garbage-collection ↩

2. Memory leak when body is not used node-fetch/node-fetch#83 ↩

[changeset-bot]

🦋 Changeset detected

Latest commit: d4ab440

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
@vercel/edge-config	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[dferber90]

This is simply brilliant, thank you a lot for bringing this to our attention and fixing it

[dferber90]

I'm asking around internally to see if we need to apply a similar workaround in index.edge.ts which runs in our Edge Functions / Edge Runtime and I'll release v0.1.7 once that's clarified.