diff --git a/.gitignore b/.gitignore index 9043d16..bcd4cf8 100644 --- a/.gitignore +++ b/.gitignore @@ -1,4 +1,5 @@ /.bundle/ +/vendor/bundle/ /.yardoc /Gemfile.lock /_yardoc/ diff --git a/lib/omniauth-mlh/version.rb b/lib/omniauth-mlh/version.rb index b61c8ad..474dcd5 100644 --- a/lib/omniauth-mlh/version.rb +++ b/lib/omniauth-mlh/version.rb @@ -2,6 +2,6 @@ module OmniAuth module MLH - VERSION = '4.1.0' + VERSION = '4.1.1' end end diff --git a/lib/omniauth/strategies/mlh.rb b/lib/omniauth/strategies/mlh.rb index 14cf495..6d13830 100644 --- a/lib/omniauth/strategies/mlh.rb +++ b/lib/omniauth/strategies/mlh.rb @@ -30,9 +30,9 @@ class MLH < OmniAuth::Strategies::OAuth2 # :nodoc: option :name, :mlh option :client_options, { - site: 'https://my.mlh.io', - authorize_url: 'oauth/authorize', - token_url: 'oauth/token', + site: 'https://www.mlh.com', + authorize_url: '/oauth/authorize', + token_url: 'https://api.mlh.com/v4/oauth/token', auth_scheme: :request_body # Change from basic auth to request body } diff --git a/spec/omni_auth/mlh_spec.rb b/spec/omni_auth/mlh_spec.rb index 5c20e2a..41f6fa8 100644 --- a/spec/omni_auth/mlh_spec.rb +++ b/spec/omni_auth/mlh_spec.rb @@ -5,7 +5,7 @@ RSpec.describe OmniAuth::MLH do it 'has a version number' do expect(OmniAuth::MLH::VERSION).not_to be_nil - expect(OmniAuth::MLH::VERSION).to eq('4.1.0') + expect(OmniAuth::MLH::VERSION).to eq('4.1.1') end it 'loads the MLH strategy' do diff --git a/spec/omni_auth/strategies/mlh_spec.rb b/spec/omni_auth/strategies/mlh_spec.rb index 7791aad..72476d3 100644 --- a/spec/omni_auth/strategies/mlh_spec.rb +++ b/spec/omni_auth/strategies/mlh_spec.rb @@ -12,6 +12,13 @@ allow(strategy).to receive(:access_token).and_return(access_token) end + describe 'client options' do + it 'uses the current MLH OAuth authorize and token endpoints' do + expect(strategy.options.client_options.authorize_url).to eq('/oauth/authorize') + expect(strategy.options.client_options.token_url).to eq('https://api.mlh.com/v4/oauth/token') + end + end + shared_context 'with oauth response' do |response_data| let(:oauth_response) do instance_double(OAuth2::Response,