Merge pull request #149 from sebringrose/hotfixes

fix: seperate hash/response in ssrHandler
sejori · Mar 26, 2023 · e4819ba · e4819ba
2 parents b728e26 + 123daf1
commit e4819ba
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/handlers/ssr.ts b/handlers/ssr.ts
@@ -19,15 +19,13 @@ export const ssrHandler = (render: Render, opts: ssrHandlerOptions = {}): Handle
  if (!opts.crypto) {
  opts.crypto = new Crypto(crypto.randomUUID())
  }
-
- const HTML = await render(ctx) 
- const hashString = await opts.crypto.hash(HTML)
 
+ const hashString = await opts.crypto.hash(await render(ctx))
  const headers = new Headers({
  "Content-Type": "text/html; charset=utf-8",
  "ETag": hashString
  })
  if (opts.headers) mergeHeaders(headers, opts.headers)
 
- return new Response(HTML, { headers })
+ return new Response(await render(ctx), { headers })
 }