omniauth 1.0.3 → 1.1.0

data/Gemfile

@@ -1,4 +1,17 @@
 source 'https://rubygems.org'
 
+gem 'growl'
+gem 'guard'
+gem 'guard-bundler'
+gem 'guard-rspec'
 gem 'rack', '~> 1.4'
+gem 'rb-fsevent'
+#gem 'plymouth'
+#gem 'pry'
+#gem 'pry-nav'
+
+platforms :jruby do
+  gem 'jruby-openssl'
+end
+
 gemspec

data/README.md

@@ -120,6 +120,16 @@ environment information on the callback request. It is entirely up to
 you how you want to implement the particulars of your application's
 authentication flow.
 
+## Logging
+
+OmniAuth supports a configurable logger. By default, OmniAuth will log 
+to `STDOUT` but you can configure this using `OmniAuth.config.logger`:
+
+```ruby
+# Rails application example
+OmniAuth.config.logger = Rails.logger
+```
+
 ## <a name="resources"></a>Resources
 
 The [OmniAuth Wiki](https://github.com/intridea/omniauth/wiki) has
@@ -129,8 +139,8 @@ OmniAuth, how it works, and how to use it.
 
 ## <a name="versions"></a>Supported Ruby Versions
 
-OmniAuth is tested under 1.8.7, 1.9.2, 1.9.3, JRuby, Rubinius (1.8
-mode), and Ruby Enterprise Edition.
+OmniAuth is tested under 1.8.7, 1.9.2, 1.9.3, JRuby (1.8 mode), and Rubinius
+(1.8 and 1.9 modes).
 
 ## <a name="license"></a>License
 

data/lib/omniauth.rb

@@ -1,7 +1,10 @@
 require 'rack'
 require 'singleton'
+require 'logger'
 
 module OmniAuth
+  class Error < StandardError; end
+  
   module Strategies
     autoload :Developer, 'omniauth/strategies/developer'
   end
@@ -11,6 +14,7 @@ module OmniAuth
   autoload :Test,     'omniauth/test'
   autoload :Form,     'omniauth/form'
   autoload :AuthHash, 'omniauth/auth_hash'
+  autoload :FailureEndpoint, 'omniauth/failure_endpoint'
 
   def self.strategies
     @@strategies ||= []
@@ -19,16 +23,19 @@ module OmniAuth
   class Configuration
     include Singleton
 
+    def self.default_logger
+      logger = Logger.new(STDOUT)
+      logger.progname = "omniauth"
+      logger
+    end
+
     @@defaults = {
       :camelizations => {},
       :path_prefix => '/auth',
-      :on_failure => Proc.new do |env|
-        message_key = env['omniauth.error.type']
-        new_path = "#{env['SCRIPT_NAME']}#{OmniAuth.config.path_prefix}/failure?message=#{message_key}"
-        [302, {'Location' => new_path, 'Content-Type'=> 'text/html'}, []]
-      end,
+      :on_failure => OmniAuth::FailureEndpoint,
       :form_css => Form::DEFAULT_CSS,
       :test_mode => false,
+      :logger => default_logger,
       :allowed_request_methods => [:get, :post],
       :mock_auth => {
         :default => AuthHash.new(
@@ -89,7 +96,7 @@ module OmniAuth
     end
 
     attr_writer :on_failure
-    attr_accessor :path_prefix, :allowed_request_methods, :form_css, :test_mode, :mock_auth, :full_host, :camelizations
+    attr_accessor :path_prefix, :allowed_request_methods, :form_css, :test_mode, :mock_auth, :full_host, :camelizations, :logger
   end
 
   def self.config
@@ -100,6 +107,10 @@ module OmniAuth
     yield config
   end
 
+  def self.logger
+    config.logger
+  end
+
   def self.mock_auth_for(provider)
     config.mock_auth[provider.to_sym] || config.mock_auth[:default]
   end

data/lib/omniauth/builder.rb

@@ -23,6 +23,11 @@ module OmniAuth
       OmniAuth.configure(&block)
     end
 
+    def options(options = false)
+      return @options || {} if options == false
+      @options = options
+    end
+
     def provider(klass, *args, &block)
       if klass.is_a?(Class)
         middleware = klass
@@ -34,6 +39,7 @@ module OmniAuth
         end
       end
 
+      args.last.is_a?(Hash) ? args.push(options.merge(args.pop)) : args.push(options)
       use middleware, *args, &block
     end
 

data/lib/omniauth/failure_endpoint.rb

@@ -0,0 +1,44 @@
+module OmniAuth
+  # This simple Rack endpoint that serves as the default
+  # 'failure' mechanism for OmniAuth. If a strategy fails for
+  # any reason this endpoint will be invoked. The default behavior
+  # is to redirect to `/auth/failure` except in the case of
+  # a development `RACK_ENV`, in which case an exception will
+  # be raised.
+  class FailureEndpoint
+    attr_reader :env
+
+    def self.call(env)
+      new(env).call
+    end
+
+    def initialize(env)
+      @env = env
+    end
+
+    def call
+      raise_out! if ENV['RACK_ENV'].to_s == 'development'
+      redirect_to_failure
+    end
+
+    def raise_out!
+      raise env['omniauth.error'] || OmniAuth::Error.new(env['omniauth.error.type'])
+    end
+
+    def redirect_to_failure
+      message_key = env['omniauth.error.type']
+      new_path = "#{env['SCRIPT_NAME']}#{OmniAuth.config.path_prefix}/failure?message=#{message_key}#{origin_query_param}#{strategy_name_query_param}"
+      Rack::Response.new(["302 Moved"], 302, 'Location' => new_path).finish
+    end
+
+    def strategy_name_query_param
+      return "" unless env['omniauth.error.strategy']
+      "&strategy=#{env['omniauth.error.strategy'].name}"
+    end
+
+    def origin_query_param
+      return "" unless env['omniauth.origin']
+      "&origin=#{Rack::Utils.escape(env['omniauth.origin'])}"
+    end
+  end
+end

data/lib/omniauth/form.rb

@@ -151,6 +151,7 @@ module OmniAuth
       <!DOCTYPE html>
       <html>
       <head>
+        <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
         <title>#{title}</title>
         #{css}
         #{header_info}

data/lib/omniauth/strategy.rb

@@ -142,6 +142,15 @@ module OmniAuth
       "#<#{self.class.to_s}>"
     end
 
+    # Direct access to the OmniAuth logger, automatically prefixed
+    # with this strategy's name.
+    #
+    # @example
+    #   log :warn, "This is a warning."
+    def log(level, message)
+      OmniAuth.logger.send(level, "(#{name}) #{message}")
+    end
+
     # Duplicates this instance and runs #call! on it.
     # @param [Hash] The Rack environment.
     def call(env)
@@ -178,12 +187,16 @@ module OmniAuth
     def request_call
       setup_phase
 
+      log :info, "Request phase initiated."
+
       #store query params from the request url, extracted in the callback_phase
       session['omniauth.params'] = request.params
 
       if options.form.respond_to?(:call)
+        log :info, "Rendering form from supplied Rack endpoint."
         options.form.call(env)
       elsif options.form
+        log :info, "Rendering form from underlying application."
         call_app!
       else
         if request.params['origin']
@@ -198,6 +211,8 @@ module OmniAuth
     # Performs the steps necessary to run the callback phase of a strategy.
     def callback_call
       setup_phase
+
+      log :info, "Callback phase initiated."
       @env['omniauth.origin'] = session.delete('omniauth.origin')
       @env['omniauth.origin'] = nil if env['omniauth.origin'] == ''
       @env['omniauth.params'] = session.delete('omniauth.params') || {}
@@ -211,11 +226,19 @@ module OmniAuth
     end
 
     def on_request_path?
-      on_path?(request_path)
+      if options.request_path.respond_to?(:call)
+        options.request_path.call(env)
+      else
+        on_path?(request_path)
+      end
     end
 
     def on_callback_path?
-      on_path?(callback_path)
+      if options.callback_path.respond_to?(:call)
+        options.callback_path.call(env)
+      else
+        on_path?(callback_path)
+      end
     end
 
     def on_path?(path)
@@ -237,9 +260,6 @@ module OmniAuth
 
     def mock_request_call
       setup_phase
-      if response = call_through_to_app
-        return response
-      end
 
       if request.params['origin']
         @env['rack.session']['omniauth.origin'] = request.params['origin']
@@ -269,8 +289,10 @@ module OmniAuth
     # underlying application. This will default to `/auth/:provider/setup`.
     def setup_phase
       if options[:setup].respond_to?(:call)
+        log :info, "Setup endpoint detected, running now."
         options[:setup].call(env)
       elsif options.setup?
+        log :info, "Calling through to underlying application for setup."
         setup_env = env.merge('PATH_INFO' => setup_path, 'REQUEST_METHOD' => 'GET')
         call_app!(setup_env)
       end
@@ -336,12 +358,22 @@ module OmniAuth
       options[:path_prefix] || OmniAuth.config.path_prefix
     end
 
+    def custom_path(kind)
+      if options[kind].respond_to?(:call)
+        result = options[kind].call(env) 
+        return nil unless result.is_a?(String)
+        result
+      else
+        options[kind]
+      end
+    end
+
     def request_path
-      options[:request_path] || "#{path_prefix}/#{name}"
+      options[:request_path].is_a?(String) ? options[:request_path] : "#{path_prefix}/#{name}"
     end
 
     def callback_path
-      options[:callback_path] || "#{path_prefix}/#{name}/callback"
+      options[:callback_path].is_a?(String) ? options[:callback_path] : (custom_path(:request_path) || "#{path_prefix}/#{name}/callback")
     end
 
     def setup_path
@@ -356,14 +388,6 @@ module OmniAuth
       request.query_string.empty? ? "" : "?#{request.query_string}"
     end
 
-    def call_through_to_app
-      status, headers, body = *call_app!
-      session['query_params'] = Rack::Request.new(env).params
-      @response = Rack::Response.new(body, status, headers)
-
-      status == 404 ? nil : @response.finish
-    end
-
     def call_app!(env = @env)
       @app.call(env)
     end
@@ -424,6 +448,12 @@ module OmniAuth
       self.env['omniauth.error.type'] = message_key.to_sym
       self.env['omniauth.error.strategy'] = self
 
+      if exception
+        log :error, "Authentication failure! #{message_key}: #{exception.class.to_s}, #{exception.message}"
+      else
+        log :error, "Authentication failure! #{message_key} encountered."
+      end
+
       OmniAuth.config.on_failure.call(self.env)
     end
 

data/lib/omniauth/version.rb

@@ -1,3 +1,3 @@
 module OmniAuth
-  VERSION = "1.0.3"
+  VERSION = "1.1.0"
 end

data/omniauth.gemspec

@@ -10,14 +10,9 @@ Gem::Specification.new do |gem|
   gem.add_runtime_dependency 'rack'
   gem.add_runtime_dependency 'hashie', '~> 1.2'
 
-  gem.add_development_dependency 'growl'
-  gem.add_development_dependency 'guard'
-  gem.add_development_dependency 'guard-bundler'
-  gem.add_development_dependency 'guard-rspec'
   gem.add_development_dependency 'simplecov'
   gem.add_development_dependency 'rack-test'
   gem.add_development_dependency 'rake'
-  gem.add_development_dependency 'rb-fsevent'
   gem.add_development_dependency 'rdiscount'
   gem.add_development_dependency 'rspec', '~> 2.8'
   gem.add_development_dependency 'yard'

data/spec/omniauth/builder_spec.rb

@@ -17,7 +17,7 @@ describe OmniAuth::Builder do
       end }.should_not raise_error
     end
 
-    it "should raise a helpful LoadError messgae if it can't find the class" do
+    it "should raise a helpful LoadError message if it can't find the class" do
       expect {
         OmniAuth::Builder.new(nil) do
           provider :lorax
@@ -25,4 +25,24 @@ describe OmniAuth::Builder do
       }.to raise_error(LoadError, "Could not find matching strategy for :lorax. You may need to install an additional gem (such as omniauth-lorax).")
     end
   end
+
+  describe '#options' do
+    it 'should merge provided options in' do
+      k = Class.new
+      b = OmniAuth::Builder.new(nil)
+      b.should_receive(:use).with(k, :foo => 'bar', :baz => 'tik')
+
+      b.options :foo => 'bar'
+      b.provider k, :baz => 'tik'
+    end
+
+    it 'should add an argument if no options are provided' do
+      k = Class.new
+      b = OmniAuth::Builder.new(nil)
+      b.should_receive(:use).with(k, :foo => 'bar')
+
+      b.options :foo => 'bar'
+      b.provider k
+    end
+  end
 end

data/spec/omniauth/failure_endpoint_spec.rb

@@ -0,0 +1,53 @@
+require 'spec_helper'
+
+describe OmniAuth::FailureEndpoint do
+  subject{ OmniAuth::FailureEndpoint }
+
+  context 'development' do
+    before do
+      @rack_env = ENV['RACK_ENV']
+      ENV['RACK_ENV'] = 'development'
+    end
+
+    it 'should raise out the error' do
+      expect do
+        subject.call('omniauth.error' => StandardError.new("Blah"))
+      end.to raise_error(StandardError, "Blah")
+    end
+
+    it 'should raise out an OmniAuth::Error if no omniauth.error is set' do
+      expect{ subject.call('omniauth.error.type' => 'example') }.to raise_error(OmniAuth::Error, "example")
+    end
+
+    after do
+      ENV['RACK_ENV'] = @rack_env
+    end
+  end
+
+  context 'non-development' do
+    let(:env){ {'omniauth.error.type' => 'invalid_request',
+                'omniauth.error.strategy' => ExampleStrategy.new({}) } }
+
+    it 'should be a redirect' do
+      status, head, body = *subject.call(env)
+      status.should == 302
+    end
+
+    it 'should include the SCRIPT_NAME' do
+      status, head, body = *subject.call(env.merge('SCRIPT_NAME' => '/random'))
+      head['Location'].should == '/random/auth/failure?message=invalid_request&strategy=test'
+    end
+
+    it 'should respect configured path prefix' do
+      OmniAuth.config.stub(:path_prefix => '/boo')
+      status, head, body = *subject.call(env)
+      head["Location"].should == '/boo/failure?message=invalid_request&strategy=test'
+    end
+
+    it 'should include the origin (escaped) if one is provided' do
+      env.merge! 'omniauth.origin' => '/origin-example'
+      status, head, body = *subject.call(env)
+      head['Location'].should be_include('&origin=%2Forigin-example')
+    end
+  end
+end

data/spec/omniauth/strategy_spec.rb

@@ -1,23 +1,4 @@
-require File.expand_path('../../spec_helper', __FILE__)
-
-class ExampleStrategy
-  include OmniAuth::Strategy
-  option :name, 'test'
-  def call(env); self.call!(env) end
-  attr_reader :last_env
-  def request_phase
-    @fail = fail!(options[:failure]) if options[:failure]
-    @last_env = env
-    return @fail if @fail
-    raise "Request Phase"
-  end
-  def callback_phase
-    @fail = fail!(options[:failure]) if options[:failure]
-    @last_env = env
-    return @fail if @fail
-    raise "Callback Phase"
-  end
-end
+require 'spec_helper'
 
 def make_env(path = '/auth/test', props = {})
   {
@@ -365,6 +346,23 @@ describe OmniAuth::Strategy do
       end
     end
 
+    context 'dynamic paths' do
+      it 'should run the request phase if the custom request path evaluator is truthy' do
+        @options = {:request_path => lambda{|env| true}}
+        lambda{ strategy.call(make_env('/asoufibasfi')) }.should raise_error("Request Phase")
+      end
+
+      it 'should run the callback phase if the custom callback path evaluator is truthy' do
+        @options = {:callback_path => lambda{|env| true}}
+        lambda{ strategy.call(make_env('/asoufiasod')) }.should raise_error("Callback Phase")
+      end
+
+      it 'should provide a custom callback path if request_path evals to a string' do
+        strategy_instance = fresh_strategy.new(nil, :request_path => lambda{|env| "/auth/boo/callback/22" })
+        strategy_instance.callback_path.should == '/auth/boo/callback/22'
+      end      
+    end
+
     context 'custom paths' do
       it 'should use a custom request_path if one is provided' do
         @options = {:request_path => '/awesome'}
@@ -477,6 +475,11 @@ describe OmniAuth::Strategy do
     end
 
     context 'test mode' do
+      let(:app) do
+        # In test mode, the underlying app shouldn't be called on request phase.
+        lambda { |env| [404, {"Content-Type" => "text/html"}, []] }
+      end
+
       before do
         OmniAuth.config.test_mode = true
       end
@@ -509,7 +512,7 @@ describe OmniAuth::Strategy do
       end
 
       it 'should be case insensitive on callback path' do
-        strategy.call(make_env('/AUTH/TeSt/CaLlBAck')).should == strategy.call(make_env('/auth/test/callback'))
+        strategy.call(make_env('/AUTH/TeSt/CaLlBAck')).first.should == strategy.call(make_env('/auth/test/callback')).first
       end
 
       it 'should maintain query string parameters' do
@@ -632,4 +635,4 @@ describe OmniAuth::Strategy do
       OmniAuth.config.test_mode = false
     end
   end
-end
+end

data/spec/omniauth_spec.rb

@@ -76,6 +76,13 @@ describe OmniAuth do
     end
   end
 
+  describe '.logger' do
+    it 'should call through to the configured logger' do
+      OmniAuth.stub(:config => mock(:logger => "foo"))
+      OmniAuth.logger.should == "foo"
+    end
+  end
+
   describe '::Utils' do
     describe '.deep_merge' do
       it 'should combine hashes' do

data/spec/spec_helper.rb

@@ -8,8 +8,28 @@ require 'rack/test'
 require 'omniauth'
 require 'omniauth/test'
 
+OmniAuth.config.logger = Logger.new("/dev/null")
+
 RSpec.configure do |config|
   config.include Rack::Test::Methods
   config.extend  OmniAuth::Test::StrategyMacros, :type => :strategy
 end
 
+class ExampleStrategy
+  include OmniAuth::Strategy
+  option :name, 'test'
+  def call(env); self.call!(env) end
+  attr_reader :last_env
+  def request_phase
+    @fail = fail!(options[:failure]) if options[:failure]
+    @last_env = env
+    return @fail if @fail
+    raise "Request Phase"
+  end
+  def callback_phase
+    @fail = fail!(options[:failure]) if options[:failure]
+    @last_env = env
+    return @fail if @fail
+    raise "Callback Phase"
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: omniauth
 version: !ruby/object:Gem::Version
-  version: 1.0.3
+  version: 1.1.0
   prerelease: 
 platform: ruby
 authors:
@@ -10,11 +10,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-03-06 00:00:00.000000000 Z
+date: 2012-04-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rack
-  requirement: &70129876115300 !ruby/object:Gem::Requirement
+  requirement: &70235277856040 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -22,10 +22,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70129876115300
+  version_requirements: *70235277856040
 - !ruby/object:Gem::Dependency
   name: hashie
-  requirement: &70129876114800 !ruby/object:Gem::Requirement
+  requirement: &70235277853920 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -33,54 +33,10 @@ dependencies:
         version: '1.2'
   type: :runtime
   prerelease: false
-  version_requirements: *70129876114800
-- !ruby/object:Gem::Dependency
-  name: growl
-  requirement: &70129876114380 !ruby/object:Gem::Requirement
-    none: false
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: *70129876114380
-- !ruby/object:Gem::Dependency
-  name: guard
-  requirement: &70129876113920 !ruby/object:Gem::Requirement
-    none: false
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: *70129876113920
-- !ruby/object:Gem::Dependency
-  name: guard-bundler
-  requirement: &70129876113500 !ruby/object:Gem::Requirement
-    none: false
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: *70129876113500
-- !ruby/object:Gem::Dependency
-  name: guard-rspec
-  requirement: &70129876113080 !ruby/object:Gem::Requirement
-    none: false
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: *70129876113080
+  version_requirements: *70235277853920
 - !ruby/object:Gem::Dependency
   name: simplecov
-  requirement: &70129876112660 !ruby/object:Gem::Requirement
+  requirement: &70235277853000 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -88,10 +44,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70129876112660
+  version_requirements: *70235277853000
 - !ruby/object:Gem::Dependency
   name: rack-test
-  requirement: &70129876112240 !ruby/object:Gem::Requirement
+  requirement: &70235277851980 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -99,21 +55,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70129876112240
+  version_requirements: *70235277851980
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: &70129876111820 !ruby/object:Gem::Requirement
-    none: false
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: *70129876111820
-- !ruby/object:Gem::Dependency
-  name: rb-fsevent
-  requirement: &70129876111400 !ruby/object:Gem::Requirement
+  requirement: &70235277850580 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -121,10 +66,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70129876111400
+  version_requirements: *70235277850580
 - !ruby/object:Gem::Dependency
   name: rdiscount
-  requirement: &70129876110980 !ruby/object:Gem::Requirement
+  requirement: &70235277848860 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -132,10 +77,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70129876110980
+  version_requirements: *70235277848860
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &70129876110480 !ruby/object:Gem::Requirement
+  requirement: &70235277846620 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -143,10 +88,10 @@ dependencies:
         version: '2.8'
   type: :development
   prerelease: false
-  version_requirements: *70129876110480
+  version_requirements: *70235277846620
 - !ruby/object:Gem::Dependency
   name: yard
-  requirement: &70129870763900 !ruby/object:Gem::Requirement
+  requirement: &70235277845920 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -154,7 +99,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70129870763900
+  version_requirements: *70235277845920
 description: A generalized Rack framework for multiple-provider authentication.
 email:
 - michael@intridea.com
@@ -177,6 +122,7 @@ files:
 - lib/omniauth.rb
 - lib/omniauth/auth_hash.rb
 - lib/omniauth/builder.rb
+- lib/omniauth/failure_endpoint.rb
 - lib/omniauth/form.rb
 - lib/omniauth/strategies/developer.rb
 - lib/omniauth/strategy.rb
@@ -188,6 +134,7 @@ files:
 - omniauth.gemspec
 - spec/omniauth/auth_hash_spec.rb
 - spec/omniauth/builder_spec.rb
+- spec/omniauth/failure_endpoint_spec.rb
 - spec/omniauth/form_spec.rb
 - spec/omniauth/strategies/developer_spec.rb
 - spec/omniauth/strategy_spec.rb
@@ -220,6 +167,7 @@ summary: A generalized Rack framework for multiple-provider authentication.
 test_files:
 - spec/omniauth/auth_hash_spec.rb
 - spec/omniauth/builder_spec.rb
+- spec/omniauth/failure_endpoint_spec.rb
 - spec/omniauth/form_spec.rb
 - spec/omniauth/strategies/developer_spec.rb
 - spec/omniauth/strategy_spec.rb