refactor(request): toLowerCase() is unnecessary for req.header() (#…

…3880)

src/request.test.ts

@@ -161,6 +161,20 @@ describe('headers', () => {
     const foo = req.header('foo')
     expect(foo).toEqual('')
   })
+
+  test('Keys of the arguments for req.header() are not case-sensitive', () => {
+    const req = new HonoRequest(
+      new Request('http://localhost', {
+        headers: {
+          'Content-Type': 'application/json',
+          apikey: 'abc',
+          lowercase: 'lowercase value',
+        },
+      })
+    )
+    expect(req.header('Content-Type')).toBe('application/json')
+    expect(req.header('ApiKey')).toBe('abc')
+  })
 })
 
 const text = '{"foo":"bar"}'

src/request.ts

@@ -178,7 +178,7 @@ export class HonoRequest<P extends string = '/', I extends Input['out'] = {}> {
   header(): Record<RequestHeader | (string & CustomHeader), string>
   header(name?: string) {
     if (name) {
-      return this.raw.headers.get(name.toLowerCase()) ?? undefined
+      return this.raw.headers.get(name) ?? undefined
     }
 
     const headerData: Record<string, string | undefined> = {}