authinator 0.0.3 → 0.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: b3d82624397aabd4fc7fb6b6b851dd6537779d39
-  data.tar.gz: ff45303cba2e2fc51ccc166bc9ad4fb694d3a7d3
+  metadata.gz: 5bab186640c9c1e8c020d88808cea1d05389e743
+  data.tar.gz: fd3636d7972b4bda346d1c9b641d633f2e655053
 SHA512:
-  metadata.gz: cf8e1d9e08b9eb1b3e3532b3366975f3f070111cc5de18b30cc61ee45e4bbb5c351a540c5ab9b560376519647b4abb1e207588d4c8bef0b57e3ad11141a117bf
-  data.tar.gz: 4bca86ce9fa8a423bf222e7759f7e56b62a0a5800497e24a261e9795124f3067e62f0a24aa900170408a7856a289ab01a59918c59727e4b76e95335ee53c7fbd
+  metadata.gz: 84848419144e543347fac7faa7fe32e13b10da1ea8e282a9c32f0977dfad44e161313c310dd3114b85c65667d5764bf9c569fd843448074c67cf7635c1d55cb5
+  data.tar.gz: db8e6561e4032b1b6f7f71648d1f48219ff2b94d260a55fae5f0f237997a7d746904fe5a27e95de0ed9f5846ac60b37444d5763ff3f2d85b1baace23c608826d

data/Gemfile

@@ -16,3 +16,6 @@ group :test do
   gem 'simplecov', '>= 0.9'
   gem 'webmock'
 end
+
+# gem 'abstract_google_client', path: '/Users/abradner/foogi/google_client', require: 'google_client'
+gem 'abstract_google_client', github: 'abradner/google_client', require: 'google_client'

data/lib/authinator.rb

@@ -1,8 +1,10 @@
+require 'active_support/all'
 require 'authinator/version'
+require 'authinator/provider'
+require 'authinator/configuration'
 require 'authinator/auth_code_exchanger'
 require 'authinator/end_point_listener'
 require 'authinator/client_token_issuer'
 
 module Authinator
-  # Your code goes here...
 end

data/lib/authinator/auth_code_exchanger.rb

@@ -1,10 +1,7 @@
-# require 'omniauth/strategies/google_oauth2'
-# require 'omniauth-oauth2'
 require 'oauth2'
 
 module Authinator
   class AuthCodeExchanger
-    VALID_PROVIDERS = [:stub, :google]
     STUB_SAMPLE_TOKEN = {
       token: 'ya29.token',
       refresh_token: '1/refresh',
@@ -12,46 +9,25 @@ module Authinator
     }
 
     attr_reader :provider
+    # attr_reader :client
 
     def self.valid_providers
-      VALID_PROVIDERS
+      Authinator.configuration.providers
     end
 
-    PROVIDER_HASHES = {
-      google: {
-        client_id: 'cl_id',
-        client_secret: 'cl_sec',
-        site: 'https://accounts.google.com',
-        token_url: '/o/oauth2/token',
-      },
-      stub: {
-        client_id: 'cl_id',
-        client_secret: 'cl_sec',
-        site: 'https://example.org',
-        token_url: '/extoken',
-      },
-    }
-
-    def initialize(provider, client_options = {})
-      unless VALID_PROVIDERS.include? provider
-        fail ArgumentError,
-             "Provider #{provider} not in supported parameter list:\n" <<
-               VALID_PROVIDERS.inspect
-      end
-
+    def initialize(provider, _client_options = {})
       @provider = provider
-      build_provider_hash(client_options)
     end
 
     def site_token_url
-      @provider_hash[:site] + @provider_hash[:token_url]
+      @provider.site + @provider.token_url
     end
 
     def exchange(auth_code)
       # auth_code = params[:code]
       return if auth_code.nil? || auth_code.empty?
 
-      case @provider.to_sym
+      case @provider.name
       when :google
         exchange_with_google(auth_code)
       when :stub
@@ -61,20 +37,16 @@ module Authinator
 
   private
 
-    def build_provider_hash(client_options)
-      @provider_hash = PROVIDER_HASHES[@provider.to_sym]
-      @provider_hash[:client_id] = client_options.delete(:client_id) if client_options[:client_id]
-      @provider_hash[:client_secret] = client_options.delete(:client_secret) if client_options[:client_secret]
-    end
+    # def build_provider_hash(client_options)
+    #   @provider_hash = Authinator.configuration.provider_for[@provider.to_sym]
+    #   @provider_hash[:client_id] = client_options.delete(:client_id) if client_options[:client_id]
+    #   @provider_hash[:client_secret] = client_options.delete(:client_secret) if client_options[:client_secret]
+    # end
 
     def exchange_with_google(code)
-      provider_hash = @provider_hash.dup
-
-      client_id = provider_hash.delete(:client_id)
-      client_secret = provider_hash.delete(:client_secret)
-      @client = OAuth2::Client.new(client_id, client_secret, provider_hash)
+      @client = OAuth2::Client.new(@provider.client_id, @provider.client_secret, @provider.to_hash)
 
-      token = @client.auth_code.get_token(code)
+      token = @client.auth_code.get_token(code, redirect_uri: 'http://localhost:4200')
 
       # response = token.get('/api/resource', :params => { 'query_foo' => 'bar' })
       # response.class.name
@@ -85,8 +57,8 @@ module Authinator
 
     def exchange_with_stub(_code)
       @client = OAuth2::Client.new(
-          @provider_hash[:client_id],
-          @provider_hash[:client_secret],
+          @provider.client_id,
+          @provider.client_secret,
       )
 
       OAuth2::AccessToken.new(

data/lib/authinator/client_token_issuer.rb

@@ -1,4 +1,226 @@
 module Authinator
+  class EmailNotVerifiedError < ArgumentError
+  end
+
   class ClientTokenIssuer
+    TEMP_RAW_INFO = {
+      email_verified: true,
+      email: 'a@b.c',
+      given_name: 'first',
+      family_name: 'last',
+      profile: 'xyz',
+    }.with_indifferent_access
+
+    # Options can
+    def initialize(params, options = {})
+      @params = params.with_indifferent_access
+      provider_name = (options.delete :provider if options[:provider]) || (@params[:provider].present? ? @params[:provider].to_sym : nil)
+      unless Authinator.configuration.providers.include? provider_name
+        fail ArgumentError,
+             "Provider #{provider_name} not in supported parameter list:\n" <<
+               Authinator.configuration.providers.inspect
+      end
+
+      @provider = Authinator.configuration.provider_for(provider_name)
+
+      @auth_code = (options.delete :auth_code if options[:auth_code]) || @params['code']
+      @app_name = (options.delete :app_name if options[:app_name]) || application_name
+      @valid_applications = (options.delete :valid_applications if options[:valid_applications]) || Authinator.configuration.valid_applications
+    end
+
+    def authorize!(options = {})
+      return { error: "Invalid Application #{@app_name}" }, :bad_request unless @valid_applications.include? @app_name
+
+      handler = EndPointListener.new(auth_code: @auth_code, provider: @provider)
+
+      if handler.valid?
+        return handle(options)
+
+      else
+        # Doorkeeper's defaut behaviour when the user signs in with login/password.
+        # TODO: kill this block
+        begin
+          response = strategy.authorize
+          headers.merge! response.headers
+          self.response_body = response.body.merge(user_id: (response.token.resource_owner_id && response.token.resource_owner_id.to_s)).to_json
+          self.status = response.status
+        rescue Doorkeeper::Errors::DoorkeeperError => e
+          handle_token_exception e
+        end
+
+      end
+    end
+
+    def handle(options = {})
+      application = Doorkeeper::Application.where(name: @app_name)
+      token_issuer = AuthCodeExchanger.new @provider
+      provider_access_token = token_issuer.exchange @auth_code
+
+      begin
+        info = load_info(provider_access_token)
+      rescue EmailNotVerifiedError => _e
+        return { error: 'Cannot create a Foogi account with an unverified email address' }, :bad_request
+      end
+
+      acc = find_or_create_account_from_info(info)
+      user = acc.user
+      expires_in = options.delete(:expires_in) || 2.hours
+      store_provider_credentials!(acc, provider_access_token)
+
+      client_access_token = Doorkeeper::AccessToken.create!(
+        application_id: application,
+        resource_owner_id: user.id,
+        expires_in: expires_in,
+        use_refresh_token: true,
+      )
+
+      token_data = {
+        access_token: client_access_token.token,
+        refresh_token: client_access_token.refresh_token,
+        token_type: 'bearer',
+        expires_in: client_access_token.expires_in,
+        user_id: user.id, # TODO: remove
+        provider_access_token: provider_access_token.token,
+        provider_expires_in: provider_access_token.expires_in,
+        # provider_id_token: provider_access_token.id_token,
+      }
+
+      [token_data.to_json, :ok]
+    end
+
+    def load_info(access_token)
+      # raw_info = provider_access_token.get('https://www.googleapis.com/plus/v1/people/me/openIdConnect').parsed
+
+      raw_info = case @provider.name
+                   when :google
+                     retrieve_user_from_google access_token
+                   when :stub
+                     TEMP_RAW_INFO
+                 end
+      # raw_info = TEMP_RAW_INFO
+
+      verified_email = raw_info[:email_verified] ? raw_info[:email] : nil
+      fail EmailNotVerifiedError, 'Email not verified' unless verified_email.present?
+
+      # verified_email = raw_info['email']
+      # prune!(
+      #   # email_verified: hash['emails'].first['type'].eql?('account'),
+      #   email_verified: false,
+      #   email: raw_info['emails'].first['value'],
+      #   display_name: raw_info['displayName'],
+      #   name: raw_info['name'],
+      #   picture_url: raw_info['image']['url'],
+      #   uid: raw_info['id'],
+      #   language: raw_info['language'],
+      # )
+      # prune!(
+      #   name: raw_info['name'],
+      #   email: verified_email,
+      #   first_name: raw_info['given_name'],
+      #   last_name: raw_info['family_name'],
+      #   image: raw_info['image_url'],
+      #   uid: raw_info['sub'] || verified_email,
+      #   urls: {
+      #     @provider.name => raw_info['profile'],
+      #   },
+      # )
+      raw_info
+    end
+
+  private
+
+    def find_or_create_account_from_info(info)
+      acc = Account.find_by email: info[:email], provider: @provider.name.to_s
+      if acc.blank?
+        user = User.find_by(primary_email: info[:email]) || User.create!(
+          primary_email: info[:email], display_name: info[:display_name],
+          name: info[:name], picture_url: info[:picture_url],
+        )
+        acc = user.create_account!(
+          email: info[:email],
+          provider: @provider.name.to_s,
+          provider_uid: info[:uid],
+        )
+      end
+      acc
+    end
+
+    def store_provider_credentials!(acc, access_token)
+      acc.credential.destroy unless acc.credential.blank?
+
+      acc.create_credential!(
+        access_token: access_token.token,
+        refresh_token: access_token.refresh_token,
+        expires_at: (Time.now.utc + access_token.expires_in),
+        expires: access_token.expires_in.present? ? true : false,
+      )
+    end
+
+    def application_name
+      @params['application_name'].present? ? @params['application_name'].to_sym : nil
+    end
+
+    def prune!(hash)
+      hash.delete_if do |_, v|
+        prune!(v) if v.is_a?(Hash)
+        v.nil? || (v.respond_to?(:empty?) && v.empty?)
+      end
+    end
+
+    def retrieve_user_from_google(access_token)
+      client = GoogleClient::Builder.new('plus', 'v1', 1)
+      result = client.execute access_token, client.service.people.get, userId: 'me'
+      process_google_info JSON.parse(result.body)
+    end
+
+    def process_google_info(hash)
+      {
+        email_verified: hash['emails'].first['type'].eql?('account'),
+        email: hash['emails'].first['value'],
+        display_name: hash['displayName'],
+        name: hash['name'],
+        picture_url: hash['image']['url'],
+        uid: hash['id'],
+        language: hash['language'],
+      }.with_indifferent_access
+
+      # {
+      #   "kind" => "plus#person",
+      #   "etag" => "\"xyz-something/abc\"",
+      #   "gender" => "female",
+      #   "emails" => [
+      #     {
+      #       "value" => "info@foogi.me",
+      #       "type" => "account"
+      #     }
+      #   ],
+      #   "objectType" => "person",
+      #   "id" => "12345",
+      #   "displayName" => "Leading Foogster",
+      #   "name" => {
+      #     "familyName" => "Foogster",
+      #     "givenName" => "Leading"
+      #   },
+      #   "url" => "https://plus.google.com/+FoogiMe",
+      #   "image" => {
+      #     "url" => "https://someurl/photo.jpg?sz=50",
+      #     "isDefault" => false
+      #   },
+      #   "placesLived" => [
+      #     {
+      #       "value" => "Sydney, Australia",
+      #       "primary" => true
+      #     },
+      #     {
+      #       "value" => "San Francisco, USA"
+      #     }
+      #   ],
+      #   "isPlusUser" => true,
+      #   "language" => "en_GB",
+      #   "circledByCount" => 11225,
+      #   "verified" => false,
+      #   "domain" => "foogi.me"
+      # }
+    end
   end
 end

data/lib/authinator/configuration.rb

@@ -0,0 +1,71 @@
+module Authinator
+  # Exception to handle a missing initializer
+  # class MissingConfiguration < StandardError
+  #   def initialize
+  #     super('Configuration for authinator missing. Do you have an Authinator initializer?')
+  #   end
+  # end
+
+  # Module level methods
+  class << self
+    attr_accessor :configuration
+  end
+
+  def self.configure
+    self.configuration ||= Configuration.new
+    yield(configuration)
+  end
+
+  # def self.configuration
+  #   @config || (fail MissingConfiguration.new)
+  # end
+
+  # Configuration class
+  class Configuration
+    attr_reader :providers
+    attr_accessor :valid_applications
+
+    def initialize
+      @providers = {}
+      add_provider(
+        :stub,
+        client_id: 'cl_id',
+        client_secret: 'cl_sec',
+        site: 'https://example.org',
+        token_url: '/extoken',
+        api_key: 'api_key',
+        user_info_url: 'http://example.org/info',
+      )
+      add_provider(
+        :google,
+        site: 'https://accounts.google.com',
+        token_url: '/o/oauth2/token',
+        user_info_url: 'https://www.googleapis.com/plus/v1/people/me/openIdConnect',
+      )
+    end
+
+    def providers
+      @providers.keys
+    end
+
+    def add_provider(provider_name, options = {})
+      @providers[provider_name] = Provider.new(provider_name, options)
+    end
+
+    def add_secrets(provider_name, options = {})
+      fail(
+        ArgumentError,
+        "#{provider_name} is not a configured provider.\n" \
+        "Valid Providers:\n" <<
+        providers.to_s,
+      ) if @providers[provider_name].blank?
+
+      @providers[provider_name].add_secrets(options)
+    end
+
+    # A more abstracted way of accessing the providers
+    def provider_for(provider_name)
+      @providers[provider_name]
+    end
+  end
+end

data/lib/authinator/end_point_listener.rb

@@ -1,9 +1,8 @@
 module Authinator
   class EndPointListener
-    attr_reader :email, :provider, :auth_code, :options, :errors
+    attr_reader :provider, :auth_code, :options, :errors
 
     def initialize(hash = {})
-      @email = hash.delete :email
       @provider = hash.delete :provider
       @auth_code = hash.delete :auth_code
       @options = hash
@@ -12,36 +11,24 @@ module Authinator
 
     def valid?
       validator_presence? &&
-        validator_valid_email? &&
         validator_valid_provider?
     end
 
   private
 
     def validator_presence?
-      email_present = present? @email
       provider_present = present? @provider
       auth_code_present = present? @auth_code
 
-      return true if email_present && provider_present && auth_code_present
+      return true if provider_present && auth_code_present
       errors << 'A required param is missing'
-      errors << '"email" field missing' unless email_present
       errors << '"provider" field missing' unless provider_present
       errors << '"auth_code" field missing' unless auth_code_present
       false
     end
 
-    # Regexp lovingly borrowed from https://github.com/balexand/email_validator
-    def validator_valid_email?
-      name_validation = @options[:strict_mode] ? '-a-z0-9+._' : '^@\\s'
-      regexp = /\A\s*([#{name_validation}]{1,64})@((?:[-a-z0-9]+\.)+[a-z]{2,})\s*\z/i
-      return true if @email =~ regexp
-      errors << "Email '#{@email}' is invalid"
-      false
-    end
-
     def validator_valid_provider?
-      return true if Authinator::AuthCodeExchanger.valid_providers.include? @provider.to_sym
+      return true if Authinator.configuration.providers.include? @provider.name
       errors << "Provider '#{@provider}' is invalid"
       false
     end

data/lib/authinator/provider.rb

@@ -0,0 +1,54 @@
+module Authinator
+  class Provider
+    attr_reader :name
+    attr_accessor :client_id,
+                  :client_secret,
+                  :site,
+                  :token_url,
+                  :api_key,
+                  :user_info_url
+
+    def initialize(name, options = {})
+      @name = name
+      @client_id = options.delete :client_id
+      @client_secret = options.delete :client_secret
+      @site = options.delete :site
+      @token_url = options.delete :token_url
+      @api_key = options.delete :api_key
+      @user_info_url = options.delete :user_info_url
+    end
+
+    def add_secrets(options = {})
+      @client_id = options.delete :client_id if options[:client_id]
+      @client_secret = options.delete :client_secret if options[:client_secret]
+      @api_key = options.delete :api_key if options[:api_key]
+    end
+
+    def secrets
+      sec = {}
+      sec[:client_id] = @client_id if @client_id
+      sec[:client_secret] = @client_secret if @client_secret
+      sec[:api_key] = @api_key if @api_key
+
+      sec
+    end
+
+    def to_hash
+      {
+        name: @name,
+        client_id: @client_id,
+        client_secret: @client_secret,
+        site: @site,
+        token_url: @token_url,
+        api_key: @api_key,
+        user_info_url: @user_info_url,
+      }
+    end
+
+    def empty?
+      to_hash.empty?
+    end
+
+    alias_method :inspect, :to_hash
+  end
+end

data/lib/authinator/version.rb

@@ -1,3 +1,3 @@
 module Authinator
-  VERSION = '0.0.3'
+  VERSION = '0.1.0'
 end

data/spec/authinator/auth_code_exchanger_spec.rb

@@ -3,6 +3,11 @@ require 'json'
 
 describe Authinator::AuthCodeExchanger do
   before :all do
+    Authinator.configure do |config|
+      config.add_secrets :google,
+                         client_id: 'cl_id',
+                         client_secret: 'cl_sec'
+    end
     @token_hash = {
       access_token: 'ya29.token',
       refresh_token: '1/refresh',
@@ -27,9 +32,29 @@ describe Authinator::AuthCodeExchanger do
       body: @token_hash.to_json,
       headers: { content_type: 'application/json' },
     }
+
+    @stub_provider = Authinator::Provider.new(
+      :stub,
+      client_id: 'cl_id',
+      client_secret: 'cl_sec',
+      site: 'https://example.org',
+      token_url: '/extoken',
+      api_key: 'api_key',
+      user_info_url: 'http://example.org/info',
+    )
+    @google_provider = Authinator::Provider.new(
+      :stub,
+      client_id: 'cl_id',
+      client_secret: 'cl_sec',
+      site: 'https://example.org',
+      token_url: '/extoken',
+      api_key: 'api_key',
+      user_info_url: 'http://example.org/info',
+    )
   end
+
   it 'should correctly process a generic (stub) token' do
-    ace = Authinator::AuthCodeExchanger.new(:stub)
+    ace = Authinator::AuthCodeExchanger.new(@stub_provider)
     stub_request(:post, ace.site_token_url).
       with(body: @test_env,
            headers: @req_headers).
@@ -45,7 +70,8 @@ describe Authinator::AuthCodeExchanger do
   it 'should return an AccessToken for each provider' do
     klass = OAuth2::AccessToken
 
-    Authinator::AuthCodeExchanger.valid_providers.each do |provider|
+    Authinator::AuthCodeExchanger.valid_providers.each do |provider_name|
+      provider = Authinator.configuration.provider_for provider_name
       ace = Authinator::AuthCodeExchanger.new(provider)
       stub_request(:post, ace.site_token_url).
         with(body: @test_env,
@@ -57,7 +83,7 @@ describe Authinator::AuthCodeExchanger do
   end
 
   it 'should correctly process a google token' do
-    ace = Authinator::AuthCodeExchanger.new(:google)
+    ace = Authinator::AuthCodeExchanger.new(@google_provider)
     stub_request(:post, ace.site_token_url).
       with(body: @test_env,
            headers: @req_headers).
@@ -69,19 +95,4 @@ describe Authinator::AuthCodeExchanger do
     expect(result.refresh_token).to eq @token_hash[:refresh_token]
     expect(result.expires_in).to eq @token_hash[:expires_in]
   end
-
-  it 'should gracefully not allow unsupported providers' do
-    expect do
-      Authinator::AuthCodeExchanger.new(:some_fake_provider)
-    end.to raise_error(ArgumentError)
-  end
-
-  it 'should correctly handle client information provided as a parameter' do
-    ace = Authinator::AuthCodeExchanger.new(:google, client_id: 'new_id', client_secret: 'new_secret')
-
-    stub_request(:post, ace.site_token_url).
-      with(body: @test_env.merge(client_id: 'new_id', client_secret: 'new_secret'),
-           headers: @req_headers).
-      to_return(@req_response)
-  end
 end

data/spec/authinator/client_token_issuer_spec.rb

@@ -5,4 +5,21 @@ describe Authinator::ClientTokenIssuer do
   it 'should generate our own set of tokens for the client if the provided ones exchanged successfully'
 
   it 'should all integrate to follow a standard flow to auth the api client'
+
+  it 'should gracefully not allow unsupported providers' do
+    pending
+    expect do
+      Authinator::AuthCodeExchanger.new(:some_fake_provider)
+    end.to raise_error(ArgumentError)
+  end
+
+  it 'should correctly handle client information provided as a parameter' do
+    pending
+    ace = Authinator::AuthCodeExchanger.new(:google, client_id: 'new_id', client_secret: 'new_secret')
+
+    stub_request(:post, ace.site_token_url).
+      with(body: @test_env.merge(client_id: 'new_id', client_secret: 'new_secret'),
+           headers: @req_headers).
+      to_return(@req_response)
+  end
 end

data/spec/authinator/end_point_listener_spec.rb

@@ -2,10 +2,18 @@ require 'spec_helper'
 
 describe Authinator::EndPointListener do
   before :each do
+    @stub_provider = Authinator::Provider.new(
+      :stub,
+      client_id: 'cl_id',
+      client_secret: 'cl_sec',
+      site: 'https://example.org',
+      token_url: '/extoken',
+      api_key: 'api_key',
+      user_info_url: 'http://example.org/info',
+    )
     @creds_hash = {
-      email: 'test@foogi.me',
       auth_code: '4/auth_code',
-      provider: 'google',
+      provider: @stub_provider,
     }
   end
   it 'should accept valid-looking credentials from the client' do
@@ -25,31 +33,30 @@ describe Authinator::EndPointListener do
     expect(listener.valid?).to be_falsey
   end
   it 'should return an error unless all three fields are provided' do
-    listener1 = Authinator::EndPointListener.new(@creds_hash.dup.tap { |hs| hs.delete(:email) }) # remove email
     listener2 = Authinator::EndPointListener.new(@creds_hash.dup.tap { |hs| hs.delete(:auth_code) }) # remove ac
     listener3 = Authinator::EndPointListener.new(@creds_hash.dup.tap { |hs| hs.delete(:provider) }) # remove prov
 
-    expect(listener1.valid?).to be_falsey
     expect(listener2.valid?).to be_falsey
     expect(listener3.valid?).to be_falsey
   end
-  it 'should reject invalid emails' do
-    bad_email_hash1 = @creds_hash.merge(email: 'abc')
-    bad_email_hash2 = @creds_hash.merge(email: '')
-    bad_email_hash3 = @creds_hash.merge(email: 'a@')
 
-    listener1 = Authinator::EndPointListener.new(bad_email_hash1)
-    listener2 = Authinator::EndPointListener.new(bad_email_hash2)
-    listener3 = Authinator::EndPointListener.new(bad_email_hash3)
+  it 'should reject invalid providers' do
+    bad_provider_1 = Authinator::Provider.new(
+      :bad1,
+      client_id: 'cl_id',
+      client_secret: 'cl_sec',
+      site: 'https://example.org',
+      token_url: '/extoken',
+      api_key: 'api_key',
+      user_info_url: 'http://example.org/info',
+    )
 
-    expect(listener1.valid?).to be_falsey
-    expect(listener2.valid?).to be_falsey
-    expect(listener3.valid?).to be_falsey
-  end
+    bad_provider_2 = Authinator::Provider.new(
+      :bad2,
+    )
 
-  it 'should reject invalid providers' do
-    bad_provider_hash1 = @creds_hash.merge(provider: 'some_fake_provider')
-    bad_provider_hash2 = @creds_hash.merge(provider: '')
+    bad_provider_hash1 = @creds_hash.merge(provider: bad_provider_1)
+    bad_provider_hash2 = @creds_hash.merge(provider: bad_provider_2)
 
     listener1 = Authinator::EndPointListener.new(bad_provider_hash1)
     listener2 = Authinator::EndPointListener.new(bad_provider_hash2)
@@ -63,7 +70,6 @@ describe Authinator::EndPointListener do
     listener.valid?
     expect(listener.errors).to eq [
       'A required param is missing',
-      '"email" field missing',
       '"provider" field missing',
       '"auth_code" field missing',
     ]

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: authinator
 version: !ruby/object:Gem::Version
-  version: 0.0.3
+  version: 0.1.0
 platform: ruby
 authors:
 - Alexander Bradner
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-12-23 00:00:00.000000000 Z
+date: 2015-01-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: oauth2
@@ -115,7 +115,9 @@ files:
 - lib/authinator.rb
 - lib/authinator/auth_code_exchanger.rb
 - lib/authinator/client_token_issuer.rb
+- lib/authinator/configuration.rb
 - lib/authinator/end_point_listener.rb
+- lib/authinator/provider.rb
 - lib/authinator/version.rb
 - spec/authinator/auth_code_exchanger_spec.rb
 - spec/authinator/client_token_issuer_spec.rb