use lower-case headers: cf. rack/rack#1592

keyasuda · Jan 27, 2024 · 9236dd9 · 9236dd9
1 parent 31dfe6b
commit 9236dd9
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/gem/spec/bormashino/server_spec.rb b/gem/spec/bormashino/server_spec.rb
@@ -171,7 +171,7 @@ class MockApp < Sinatra::Base
 
       it 'sends referer' do
         expect(subject.first).to eq 302
-        expect(subject[1]['Location']).to eq 'http://example.com:0/referer'
+        expect(subject[1]['location']).to eq 'http://example.com:0/referer'
       end
     end
   end

diff --git a/npm/src/applyServerResult.js b/npm/src/applyServerResult.js
@@ -41,13 +41,13 @@ export const applyServerResult = (src, target, router) => {
       return true
 
     case 302:
-      const loc = new URL(src[1]['Location'])
+      const loc = new URL(src[1]['location'])
 
       if (loc.host == 'example.com:0') {
         const path = loc.pathname + loc.search
         router.pushState(path)
       } else {
-        location.href = src[1]['Location']
+        location.href = src[1]['location']
       }
       return false
 

diff --git a/npm/src/applyServerResult.test.js b/npm/src/applyServerResult.test.js
@@ -110,7 +110,7 @@ describe('applyServerResult', () => {
 
   describe('302 relative location', () => {
     beforeEach(() => {
-      content = [302, { Location: 'http://example.com:0/location' }, []]
+      content = [302, { location: 'http://example.com:0/location' }, []]
       ret = applyServerResult(content, target, router)
     })
 
@@ -132,13 +132,13 @@ describe('applyServerResult', () => {
       delete window.location
       window.location = { assign: jest.fn() }
 
-      content = [302, { Location: 'http://absolute.example.com/location' }, []]
+      content = [302, { location: 'http://absolute.example.com/location' }, []]
 
       ret = applyServerResult(content, target, router)
     })
 
     it('navigates to the location', () => {
-      expect(location.href).toEqual(content[1]['Location'])
+      expect(location.href).toEqual(content[1]['location'])
     })
 
     it('returns true', () => {