Streams til buffer (#89)

* Omgjør streams til buffer, siden vi ikke utnytter noen av fordelene til streams

---------

Co-authored-by: Hanne Eilertsen Olsen <hanne.eilertsen.olsen@nav.no>

src/__tests__/api/dokumenter/__snapshots__/forhåndsvisning.test.ts.snap

@@ -1,3 +1,15 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
-exports[`/api/dokumenter/[journalpostId]/[dokumentId]/forhandsvisning svarer med en liste dokumenter 1`] = `""`;
+exports[`/api/dokumenter/[journalpostId]/[dokumentId]/forhandsvisning svarer med en liste dokumenter 1`] = `
+Object {
+  "data": Array [
+    98,
+    105,
+    110,
+    195,
+    166,
+    114,
+  ],
+  "type": "Buffer",
+}
+`;

src/pages/api/dokumenter/[journalpostId]/[dokumentId]/forhandsvisning.ts

@@ -2,7 +2,6 @@ import { NextApiHandler } from "next";
 import { v4 as uuidv4 } from "uuid";
 import { withSentry } from "@sentry/nextjs";
 import { getSession } from "../../../../../lib/auth.utils";
-import { Readable, Stream } from "stream";
 import { logger } from "@navikt/next-logger";
 
 const audience = `${process.env.SAF_SELVBETJENING_CLUSTER}:teamdokumenthandtering:${process.env.SAF_SELVBETJENING_SCOPE}`;
@@ -30,7 +29,7 @@ async function hentDokument(
   return fetch(endpoint, { headers });
 }
 
-export const handleHentDokument: NextApiHandler<Stream> = async (req, res) => {
+export const handleHentDokument: NextApiHandler<Buffer> = async (req, res) => {
   const session = await getSession(req);
   if (!session.token) return res.status(401).end();
 
@@ -48,20 +47,20 @@ export const handleHentDokument: NextApiHandler<Stream> = async (req, res) => {
         dokumentResponse.headers.get("Content-Disposition") || "inline",
       );
 
-      // Vi kan ikke bruke ReadableStream direkte fra fetch her, siden NextJS trenger en
-      // node ReadableStream, og vi får ut en web ReadableStream. Går dermed veien gjennom
-      // Buffer for å oversette den ene ReadableStream'en til den andre. Vi fant feil med
-      // ReadableStream.pipe når vi oppgraderte til Node v18, som har innebygget fetch (fra web).
-      // Før hadde de polyfillet en fetch fra node.
+      res.setHeader(
+        "Content-Type",
+        dokumentResponse.headers.get("Content-Type") ||
+          "application/octet-stream",
+      );
+
+      // Vi kan ikke bruke ReadableStream direkte fra fetch her, går over til å returnere buffer i stedet.
+      // Vi fant feil med ReadableStream.pipe når vi oppgraderte til Node v18, som har innebygget fetch
+      // (fra web). Før hadde de polyfillet en fetch fra node.
       // Senere kan vi teste ut https://www.npmjs.com/package/readable-web-to-node-stream
       const arrayBuffer = await dokumentResponse.arrayBuffer();
       const buffer = Buffer.from(arrayBuffer);
-      const readable = new Readable();
-      readable._read = () => {}; // _read is required but you can noop it
-      readable.push(buffer);
-      readable.push(null);
 
-      readable.pipe(res);
+      return res.status(dokumentResponse.status).send(buffer);
     })
     .catch((errors) => {
       console.log(errors);