fix: always clean http body (#7)

Author: lucacasonato

.github/workflows/ci.yml

@@ -12,9 +12,10 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        deno-version: [v1.4.0, nightly]
+        deno-version: [v1.4.2, nightly]
         unstable: [false, true]
-        no-check: [false, true]
+        # FIXME(lucacasonato): temporarially disable no-check testing (see https://github.com/denoland/deno/issues/7709)
+        no-check: [false]
     env:
       AWS_ACCESS_KEY_ID: AKIAIOSFODNN7EXAMPLE
       AWS_SECRET_ACCESS_KEY: wJalrXUtnFEMI/K7MDENG/bPxRfiCYEXAMPLEKEY

src/bucket.ts

@@ -216,6 +216,8 @@ export class S3Bucket {
         await resp.text(),
       );
     }
+    // clean up http body
+    await resp.arrayBuffer();
     return {
       etag: JSON.parse(resp.headers.get("etag")!),
       versionId: resp.headers.get("x-amz-version-id") ?? undefined,
@@ -309,6 +311,7 @@ export class S3Bucket {
         await resp.text(),
       );
     }
+    // clean up http body
     await resp.arrayBuffer();
     return {
       etag: JSON.parse(resp.headers.get("etag")!),
@@ -331,6 +334,8 @@ export class S3Bucket {
         await resp.text(),
       );
     }
+    // clean up http body
+    await resp.arrayBuffer();
     return {
       versionID: resp.headers.get("x-amz-version-id") ?? undefined,
       deleteMarker: resp.headers.get("x-amz-delete-marker") === "true",