refactor(server): drop the null and String paths from gzipBody

server/providers/common/compress.js

@@ -54,8 +54,7 @@ export function acceptsGzip(request) {
  * @returns {Promise<Buffer|null>}
  */
 export async function gzipBody(body) {
-  if (body == null) return null;
+  const raw = Buffer.isBuffer(body) ? body : Buffer.from(body);
-  const raw = Buffer.isBuffer(body) ? body : Buffer.from(String(body));
   if (raw.byteLength < MIN_COMPRESSED_BYTES) return null;
   const compressed = await gzipAsync(raw);
   return compressed.byteLength < raw.byteLength ? compressed : null;

src/tooling/responseCompression.test.mjs

@@ -81,8 +81,6 @@ test('accept-encoding is read the way clients actually write it', () => {
 });
 
 test('a body is only compressed when compression pays for itself', async () => {
-  assert.equal(await gzipBody(null), null);
-  assert.equal(await gzipBody(undefined), null);
   assert.equal(await gzipBody(''), null);
   assert.equal(
     await gzipBody('{"states":[]}'),