Use dedicated constructor

Refs #2141
PREreview · Dec 20, 2024 · 756e956 · 756e956
1 parent ad70b73
commit 756e956
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 25 deletions.
diff --git a/src/LegacyRouter.ts b/src/LegacyRouter.ts
@@ -1,4 +1,4 @@
-import { Headers, HttpRouter, HttpServerResponse } from '@effect/platform'
+import { HttpRouter, HttpServerResponse } from '@effect/platform'
 import type { Doi } from 'doi-ts'
 import { Effect, pipe, Schema } from 'effect'
 import { format } from 'fp-ts-routing'
@@ -91,8 +91,5 @@ export const LegacyRouter = HttpRouter.fromIterable([
 ])
 
 function movedPermanently(location: string) {
-  return HttpServerResponse.empty({
-    status: StatusCodes.MOVED_PERMANENTLY,
-    headers: Headers.fromInput({ location }),
-  })
+  return HttpServerResponse.redirect(location, { status: StatusCodes.MOVED_PERMANENTLY })
 }
diff --git a/src/Router.ts b/src/Router.ts
@@ -1,6 +1,5 @@
 import {
   Cookies,
-  Headers,
   type HttpMethod,
   HttpMiddleware,
   HttpRouter,
@@ -205,16 +204,12 @@ function toHttpServerResponse(
 ): Effect.Effect<HttpServerResponse.HttpServerResponse, never, Locale | TemplatePage | ExpressConfig | PublicUrl> {
   return Effect.gen(function* () {
     if (response._tag === 'RedirectResponse') {
-      return yield* HttpServerResponse.empty({
-        status: response.status,
-        headers: Headers.fromInput({ Location: response.location.toString() }),
-      })
+      return yield* HttpServerResponse.redirect(response.location, { status: response.status })
     }
 
     if (response._tag === 'FlashMessageResponse') {
-      return yield* HttpServerResponse.empty({
+      return yield* HttpServerResponse.redirect(response.location, {
         status: StatusCodes.SEE_OTHER,
-        headers: Headers.fromInput({ Location: response.location.toString() }),
         cookies: Cookies.fromIterable([
           Cookies.unsafeMakeCookie('flash-message', response.message, { httpOnly: true, path: '/' }),
         ]),
@@ -229,10 +224,7 @@ function toHttpServerResponse(
         state: new URL(`${publicUrl.origin}${response.location}`).href,
       })
 
-      return yield* HttpServerResponse.empty({
-        status: StatusCodes.MOVED_TEMPORARILY,
-        headers: Headers.fromInput({ Location: location.href }),
-      })
+      return yield* HttpServerResponse.redirect(location, { status: StatusCodes.MOVED_TEMPORARILY })
     }
 
     const locale = yield* Locale

diff --git a/src/WebApp.ts b/src/WebApp.ts
@@ -34,8 +34,7 @@ const removeTrailingSlashes = HttpMiddleware.make(app =>
       return yield* app
     }
 
-    return yield* HttpServerResponse.empty({
-      headers: Headers.fromInput({ Location: request.url.slice(0, request.url.length - 1) }),
+    return yield* HttpServerResponse.redirect(request.url.slice(0, request.url.length - 1), {
       status: StatusCodes.MOVED_PERMANENTLY,
     })
   }),

diff --git a/test/LegacyRouter.test.ts b/test/LegacyRouter.test.ts
@@ -1,4 +1,4 @@
-import { Headers, HttpServerRequest, HttpServerResponse } from '@effect/platform'
+import { HttpServerRequest, HttpServerResponse } from '@effect/platform'
 import { test } from '@fast-check/jest'
 import { describe, expect } from '@jest/globals'
 import { Effect, TestContext } from 'effect'
@@ -67,12 +67,7 @@ describe('LegacyRouter', () => {
 
       const response = yield* Effect.provideService(_.LegacyRouter, HttpServerRequest.HttpServerRequest, request)
 
-      expect(response).toStrictEqual(
-        HttpServerResponse.empty({
-          status: StatusCodes.MOVED_PERMANENTLY,
-          headers: Headers.fromInput({ location: expected }),
-        }),
-      )
+      expect(response).toStrictEqual(HttpServerResponse.redirect(expected, { status: StatusCodes.MOVED_PERMANENTLY }))
     }).pipe(Effect.provide(TestContext.TestContext), Effect.runSync),
   )
 })