Merge pull request #98 from nov/feature/handle_unexpected_format_of_jwks_success_response

handle `fail!` in correct way

Author: nov

lib/omniauth/strategies/apple.rb

@@ -6,6 +6,12 @@
 module OmniAuth
   module Strategies
     class Apple < OmniAuth::Strategies::OAuth2
+      class JWTFetchingFailed < CallbackError
+        def initialize(error_reason = nil, error_uri = nil)
+          super :jwks_fetching_failed, error_reason, error_uri
+        end
+      end
+
       option :name, 'apple'
 
       option :client_options,
@@ -102,10 +108,10 @@ def fetch_jwks
         if res.success?
           res.body
         else
-          fail!(:jwks_fetching_failed, CallbackError.new(:jwks_fetching_failed, 'HTTP Error when fetching JWKs'))
+          raise JWTFetchingFailed.new('HTTP Error when fetching JWKs')
         end
-      rescue Faraday::Error => e
-        fail!(:jwks_fetching_failed, e)
+      rescue JWTFetchingFailed, Faraday::Error => e
+        fail!(:jwks_fetching_failed, e) and nil
       end
 
       def verify_nonce!(payload)

spec/omniauth/strategies/apple_spec.rb

@@ -265,7 +265,10 @@
 
     context 'fails nonce' do
       before(:each) do
-        expect(subject).to receive(:fail!).with(:nonce_mismatch, instance_of(OmniAuth::Strategies::OAuth2::CallbackError))
+        expect(subject).to receive(:fail!).with(
+          :nonce_mismatch,
+          instance_of(OmniAuth::Strategies::OAuth2::CallbackError)
+        ).and_return([302, {}, ''])
       end
       it 'when differs from session' do
         subject.session['omniauth.nonce'] = 'abc'
@@ -356,8 +359,8 @@
       it do
         expect(subject).to receive(:fail!).with(
           :jwks_fetching_failed,
-          instance_of(OmniAuth::Strategies::OAuth2::CallbackError)
-        )
+          instance_of(OmniAuth::Strategies::Apple::JWTFetchingFailed)
+        ).and_return([302, {}, ''])
         subject.info
       end
     end
@@ -376,7 +379,7 @@
         expect(subject).to receive(:fail!).with(
           :jwks_fetching_failed,
           instance_of(Faraday::ParsingError)
-        )
+        ).and_return([302, {}, ''])
         subject.info
       end
     end