allow URL object on c.redirect

honojs · Nov 1, 2024 · e349a00 · e349a00
1 parent ae99d86
commit e349a00
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(new URL('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
@@ -820,11 +820,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', location.toString())
     return this.newResponse(null, status ?? 302) as any
   }