Merge pull request #106 from curationexperts/metadata_options

Add OPTIONS response for info.json

app/controllers/riiif/images_controller.rb

@@ -44,6 +44,12 @@ def info
       end
     end
 
+    # See https://fetch.spec.whatwg.org/#http-access-control-allow-headers
+    def info_options
+      response.headers['Access-Control-Allow-Headers'] = 'Authorization'
+      self.response_body = ''
+    end
+
     # this is a workaround for https://github.com/rails/rails/issues/25087
     def redirect
       # This was attempted with just info_path, but it gave a NoMethodError

lib/riiif/routes.rb

@@ -26,6 +26,9 @@ def draw
             defaults: { format: 'json', model: resource },
             as: [options[:as], 'info'].compact.join('_')
 
+        match "#{route_prefix}/:id/info.json" => 'riiif/images#info_options',
+              via: [:options]
+
         # This doesn't work presently
         # get "#{route_prefix}/:id", to: redirect("#{route_prefix}/%{id}/info.json")
         get "#{route_prefix}/:id" => 'riiif/images#redirect', as: [options[:as], 'base'].compact.join('_')

spec/controllers/riiif/images_controller_spec.rb

@@ -131,6 +131,14 @@
     end
   end
 
+  describe 'info_options' do
+    it 'is successful' do
+      process :info_options, method: 'OPTIONS', params: { id: 'abcd123', format: 'json' }
+      expect(response).to be_successful
+      expect(response.headers['Access-Control-Allow-Headers']).to eq 'Authorization'
+    end
+  end
+
   describe 'info' do
     context 'the happy path' do
       let(:image) { double }