omniauth-mlh 0.2.1 → 0.4.1

checksums.yaml

@@ -1,15 +1,7 @@
 ---
-!binary "U0hBMQ==":
-  metadata.gz: !binary |-
-    N2I3YTE0OTE4MGQ4ZWFhOTAzNDRlMzRiMWI1MGZmYTRlMDE5ZGYyYQ==
-  data.tar.gz: !binary |-
-    YmMxOWNlY2I1ZGEyZmIyMTNjYTVmMGJkYWZkYjViYjcwNjg5OTU2ZQ==
+SHA256:
+  metadata.gz: 1608c1c551536f2dde7aa485c7b6f61ed4ffa9131805d930f12e65aeb09b25ee
+  data.tar.gz: fcaf372603aede88fc7623f6aa981aefea2467597907a5266ab4f3ba57f4345b
 SHA512:
-  metadata.gz: !binary |-
-    NWNjNWI5YzNlNDA4YzNkZjM2MzNlNzRkNTE5NWU5MDYzMGI3ODc2YjQ2MjZk
-    MzI1ZTAxMjQyMjgzOGM1MjYxNzI4MjUxYjdmNDYyYTQwMWUzNTg5NTNjNTI5
-    MjExMmE1YzYxMDg5NzRmNDUyMDEyZDgxNWNkNjIyZTAyODFjMzY=
-  data.tar.gz: !binary |-
-    YjNmY2NmNGE0ZDFlMzRkZWY5MGU1ZmQxMjNmYTk1ODdlY2NjMGRkMTI4MGZl
-    YmJjNTAyYjY1YTc5MzU2OTEzNzNlNGY5MTUyNGI5MTI1YjUzNGVjMmI2MWY2
-    MmIwMzRjYzFiMzNkOThkYzY5ZmIzYzNmZDExODNlMmUyMjQ5Y2U=
+  metadata.gz: fffcc909ecd51283296ed328d36c5770a6fd0e1d11e8f51f40d9abb67ba0db4ce2b5638d0e83afdbeec01d39ed1393a9dcac311bc6c99d7ae291d7066313f50d
+  data.tar.gz: 8ca616032f69b5e082b02a880a75562d1620594d8fb34c611a05041d2e7217095240a6595aa5cf503b7ed2cf1a06a931ab988c7c91e405b5e64dd29add0eda63

data/.ruby-version

@@ -0,0 +1 @@
+2.2.4

data/.travis.yml

@@ -1,3 +1,3 @@
 language: ruby
 rvm:
-  - 2.2.1
+  - 2.2.4

data/README.md

@@ -1,16 +1,18 @@
-# OmniAuth MLH
+# MLH/my-mlh-omniauth
 
-This is the official OmniAuth strategy for authenticating to My MLH. To use
-it, you'll need to sign up for an OAuth2 Application ID and Secret on the
-[My MLH Applications Page](https://my.mlh.io/oauth/applications).
+[![Build Status](https://travis-ci.org/MLH/my-mlh-omniauth.svg?branch=master)](https://travis-ci.org/MLH/my-mlh-omniauth)
 
-# Usage
+This is the official [OmniAuth](https://github.com/omniauth/omniauth) strategy for
+authenticating with [MyMLH](https://my.mlh.io). To use it, you'll need to
+[register an application](https://my.mlh.io/oauth/applications) and obtain a OAuth Application ID and Secret from MyMLH.
 
-```
-use OmniAuth::Builder do
-  provider :mlh, ENV['MLH_KEY'], ENV['MLH_SECRET']
-end
-```
+It now supports MyMLH API V3. [Read the MyMLH V3 docs here](https://my.mlh.io/docs).
+
+Once you have done so, you can follow the instructions below:
+
+## Requirements
+
+You need to have at least Ruby 2.2.4 to use this gem.
 
 ## Installation
 
@@ -28,17 +30,39 @@ Or install it yourself as:
 
     $ gem install omniauth-mlh
 
+## Usage (Rack)
+
+```ruby
+use OmniAuth::Builder do
+  provider :mlh, ENV['MY_MLH_KEY'], ENV['MY_MLH_SECRET'], scope: 'default email birthday'
+end
+```
+
+## Usage (Rails with Devise)
+
+```ruby
+# config/devise.rb
+
+Devise.setup do |config|
+  config.provider :mlh, ENV['MY_MLH_KEY'], ENV['MY_MLH_SECRET'], scope: 'default email birthday'
+end
+```
+
 ## Contributing
 
-1. Fork it ( https://github.com/mlh/omniauth-mlh/fork )
+1. Fork it ( https://github.com/mlh/my-mlh-omniauth/fork )
 2. Create your feature branch (`git checkout -b my-new-feature`)
 3. Commit your changes (`git commit -am 'Add some feature'`)
 4. Push to the branch (`git push origin my-new-feature`)
 5. Create a new Pull Request
 
+## Credit
+
+We used part of [datariot/omniauth-paypal](http://github.com/datariot/omniauth-paypal)'s code to help us test this gem.
+
 ## Questions?
 
 Have a question about the API or this library? Start by checking out the
-[official My MLH documentation](https://my.mlh.io/docs). If you still can't
+[official MyMLH documentation](https://my.mlh.io/docs). If you still can't
 find an answer, tweet at [@MLHacks](http://twitter.com/mlhacks) or drop an
-email to [hi@mlh.io](mailto://hi@mlh.io).
+email to [engineering@mlh.io](mailto:engineering@mlh.io).

data/lib/omniauth-mlh.rb

@@ -1,2 +1,3 @@
+require "active_support"
 require "omniauth-mlh/version"
 require "omniauth/strategies/mlh"

data/lib/omniauth-mlh/version.rb

@@ -1,5 +1,9 @@
 module OmniAuth
   module MLH
-    VERSION = "0.2.1"
+    MAJOR_VERSION = 0
+    MINOR_VERSION = 4
+    RELEASE_VERSION = 1
+
+    VERSION = [MAJOR_VERSION, MINOR_VERSION, RELEASE_VERSION].join('.')
   end
 end

data/lib/omniauth/strategies/mlh.rb

@@ -1,4 +1,5 @@
 require 'omniauth-oauth2'
+require 'ostruct'
 
 module OmniAuth
   module Strategies
@@ -6,37 +7,32 @@ module OmniAuth
       option :name, :mlh
 
       option :client_options, {
-        :site => 'https://my.mlh.io',
-        :authorize_url  => '/oauth/authorize',
-        :token_url => '/oauth/token'
+        :site            => 'https://my.mlh.io',
+        :authorize_path  => '/oauth/authorize',
+        :token_path      => '/oauth/token'
       }
 
-      uid { raw_info['data']['id'] }
+      uid { data[:id] }
 
       info do
-        {
-          :email                => raw_info['data']['email'],
-          :created_at           => raw_info['data']['created_at'],
-          :updated_at           => raw_info['data']['updated_at'],
-          :first_name           => raw_info['data']['first_name'],
-          :last_name            => raw_info['data']['last_name'],
-          :graduation           => raw_info['data']['graduation'],
-          :major                => raw_info['data']['major'],
-          :shirt_size           => raw_info['data']['shirt_size'],
-          :dietary_restrictions => raw_info['data']['dietary_restrictions'],
-          :special_needs        => raw_info['data']['special_needs'],
-          :date_of_birth        => raw_info['data']['date_of_birth'],
-          :gender               => raw_info['data']['gender'],
-          :phone_number         => raw_info['data']['phone_number'],
-          :school               => {
-                                  :id =>  raw_info['data']['school']['id'],
-                                  :name =>  raw_info['data']['school']['name'],
-                                }
-        }
+        data.slice(
+          :email,
+          :created_at,
+          :updated_at,
+          :first_name,
+          :last_name,
+          :level_of_study,
+          :major,
+          :date_of_birth,
+          :gender,
+          :phone_number,
+          :scopes,
+          :school
+        )
       end
 
-      def raw_info
-        @raw_info ||= access_token.get('/api/v1/user.json').parsed
+      def data
+        @data ||= access_token.get('/api/v3/user.json').parsed.deep_symbolize_keys[:data] rescue {}
       end
     end
   end

data/omniauth-mlh.gemspec

@@ -14,15 +14,19 @@ Gem::Specification.new do |spec|
   spec.homepage      = "http://github.com/mlh/omniauth-mlh"
   spec.license       = "MIT"
 
+  spec.required_ruby_version = '>= 2.2.4'
+
   spec.executables    = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
   spec.files          = `git ls-files`.split("\n")
   spec.test_files     = `git ls-files -- {test,spec,features}/*`.split("\n")
   spec.require_paths = ["lib"]
 
   spec.add_dependency 'omniauth', '~> 1.0'
-  spec.add_dependency 'omniauth-oauth2', '>= 1.1.1', '< 2.0'
+  spec.add_dependency 'omniauth-oauth2', '~> 1.3.1'
+  spec.add_dependency 'activesupport'
 
   spec.add_development_dependency 'rspec', '~> 2.7'
+  spec.add_development_dependency 'rake', '~> 10.5'
   spec.add_development_dependency 'rack-test'
   spec.add_development_dependency 'simplecov'
   spec.add_development_dependency 'webmock'

data/spec/omniauth/mlh_spec.rb

@@ -2,16 +2,35 @@ require 'spec_helper'
 
 describe OmniAuth::MLH do
   subject do
-    OmniAuth::Strategies::MLH.new({})
+    OmniAuth::Strategies::MLH.new(nil, @options || {})
   end
 
-  context "client options" do
-    it 'should have correct site' do
-      expect(subject.options.client_options.site).to eq('https://my.mlh.io')
+  it_should_behave_like 'an oauth2 strategy'
+
+  describe '#client' do
+    it 'has correct MyMLH site' do
+      expect(subject.client.site).to eq('https://my.mlh.io')
+    end
+
+    it 'has correct authorize url' do
+      expect(subject.client.options[:authorize_url]).to eq('/oauth/authorize')
+    end
+
+    it 'has correct token url' do
+      expect(subject.client.options[:token_url]).to eq('/oauth/token')
+    end
+
+    it 'runs the setup block if passed one' do
+      counter = ''
+      @options = { :setup => Proc.new { |env| counter = 'ok' } }
+      subject.setup_phase
+      expect(counter).to eq("ok")
     end
+  end
 
-    it 'should have correct authorize url' do
-      expect(subject.options.client_options.authorize_path).to eq('/oauth/authorize')
+  describe '#callback_path' do
+    it "has the correct callback path" do
+      expect(subject.callback_path).to eq('/auth/mlh/callback')
     end
   end
 end

data/spec/spec_helper.rb

@@ -1,14 +1,20 @@
 $LOAD_PATH.unshift File.expand_path('../../lib', __FILE__)
+
 require 'simplecov'
 SimpleCov.start
+
+require 'bundler/setup'
 require 'rspec'
 require 'rack/test'
 require 'webmock/rspec'
 require 'omniauth'
 require 'omniauth-mlh'
+Dir[File.expand_path('../support/**/*', __FILE__)].each { |f| require f }
 
 RSpec.configure do |config|
   config.include WebMock::API
   config.include Rack::Test::Methods
   config.extend  OmniAuth::Test::StrategyMacros, :type => :strategy
+
+  OmniAuth.config.test_mode = true
 end

data/spec/support/shared_examples.rb

@@ -0,0 +1,39 @@
+# Credit: https://github.com/datariot/omniauth-paypal/blob/master/spec/support/shared_examples.rb
+# NOTE it would be useful if this lived in omniauth-oauth2 eventually
+
+shared_examples 'an oauth2 strategy' do
+  describe '#client' do
+    it 'should be initialized with symbolized client_options' do
+      @options = { :client_options => { 'authorize_url' => 'https://example.com' } }
+      expect(subject.client.options[:authorize_url]).to eq('https://example.com')
+    end
+  end
+
+  describe '#authorize_params' do
+    it 'should include any authorize params passed in the :authorize_params option' do
+      @options = { :authorize_params => { :foo => 'bar', :baz => 'zip' } }
+      expect(subject.authorize_params['foo']).to eq('bar')
+      expect(subject.authorize_params['baz']).to eq('zip')
+    end
+
+    it 'should include top-level options that are marked as :authorize_options' do
+      @options = { :authorize_options => [:scope, :foo], :scope => 'bar', :foo => 'baz' }
+      expect(subject.authorize_params['scope']).to eq('bar')
+      expect(subject.authorize_params['foo']).to eq('baz')
+    end
+  end
+
+  describe '#token_params' do
+    it 'should include any token params passed in the :token_params option' do
+      @options = { :token_params => { :foo => 'bar', :baz => 'zip' } }
+      expect(subject.token_params['foo']).to eq('bar')
+      expect(subject.token_params['baz']).to eq('zip')
+    end
+
+    it 'should include top-level options that are marked as :token_options' do
+      @options = { :token_options => [:scope, :foo], :scope => 'bar', :foo => 'baz' }
+      expect(subject.token_params['scope']).to eq('bar')
+      expect(subject.token_params['foo']).to eq('baz')
+    end
+  end
+end

metadata

@@ -1,103 +1,125 @@
 --- !ruby/object:Gem::Specification
 name: omniauth-mlh
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.4.1
 platform: ruby
 authors:
 - Swift
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2015-12-13 00:00:00.000000000 Z
+date: 2020-08-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: omniauth
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '1.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '1.0'
 - !ruby/object:Gem::Dependency
   name: omniauth-oauth2
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.1.1
-    - - <
-      - !ruby/object:Gem::Version
-        version: '2.0'
+        version: 1.3.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.1.1
-    - - <
+        version: 1.3.1
+- !ruby/object:Gem::Dependency
+  name: activesupport
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '2.0'
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '2.7'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '2.7'
+- !ruby/object:Gem::Dependency
+  name: rake
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '10.5'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '10.5'
 - !ruby/object:Gem::Dependency
   name: rack-test
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: simplecov
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: webmock
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 description: Official OmniAuth strategy for My MLH.
@@ -107,9 +129,10 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- .gitignore
-- .rspec
-- .travis.yml
+- ".gitignore"
+- ".rspec"
+- ".ruby-version"
+- ".travis.yml"
 - Gemfile
 - LICENSE.txt
 - README.md
@@ -120,30 +143,31 @@ files:
 - omniauth-mlh.gemspec
 - spec/omniauth/mlh_spec.rb
 - spec/spec_helper.rb
+- spec/support/shared_examples.rb
 homepage: http://github.com/mlh/omniauth-mlh
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ! '>='
+  - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 2.2.4
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ! '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.2.2
-signing_key: 
+rubygems_version: 3.0.8
+signing_key:
 specification_version: 4
 summary: Official OmniAuth strategy for My MLH.
 test_files:
 - spec/omniauth/mlh_spec.rb
 - spec/spec_helper.rb
+- spec/support/shared_examples.rb