feat(context): allow URL object on c.redirect() (#3609)

* feat(context): allow URL object on `c.redirect()` * Update src/context.ts Co-authored-by: EdamAmex <[email protected]> --------- Co-authored-by: EdamAmex <[email protected]>
honojs · Nov 1, 2024 · 01277aa · 01277aa
1 parent 224ec36
commit 01277aa
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/src/context.test.ts b/src/context.test.ts
@@ -89,6 +89,12 @@ describe('Context', () => {
     expect(res.headers.get('Location')).toBe('https://example.com/destination')
   })
 
+  it('c.redirect() w/ URL', async () => {
+    const res = c.redirect(new URL('/destination', 'https://example.com'))
+    expect(res.status).toBe(302)
+    expect(res.headers.get('Location')).toBe('https://example.com/destination')
+  })
+
   it('c.header()', async () => {
     c.header('X-Foo', 'Bar')
     const res = c.body('Hi')

diff --git a/src/context.ts b/src/context.ts
@@ -819,11 +819,11 @@ export class Context<
    * ```
    */
   redirect = <T extends RedirectStatusCode = 302>(
-    location: string,
+    location: string | URL,
     status?: T
   ): Response & TypedResponse<undefined, T, 'redirect'> => {
     this.#headers ??= new Headers()
-    this.#headers.set('Location', location)
+    this.#headers.set('Location', String(location))
     return this.newResponse(null, status ?? 302) as any
   }