omniauth-pandadoc 1.0.0rc2 → 1.0.0rc3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d0caba7eb011abc0fb91e4c8fd0170baa160c7c95ed41595e2b7d96c8f4270c6
-  data.tar.gz: 68952275e28e1ae676431abd94ddbb3e3d9cab50c9fd15689379655439cf9426
+  metadata.gz: 3eed4e219f8e70d1982f043aab2b31f9939fc0e426a55461d545503559019b05
+  data.tar.gz: 6b29e9cc520aa91faf49d07e8c4a4871fb2f81c1f2872bd0edacc940f7765197
 SHA512:
-  metadata.gz: 324525da59bb3b357d0e705732f8bdaea4a3be3cbe7b626d7f60a488c6ee4e4d6e75eeba2a8a731f070943b8e6903340a6c38d42a8f54291dbd245c97acea89b
-  data.tar.gz: 7bc6a67d8721d3d5bb7603d5bee21e20d83d8750ecdf90150ece6d4f3767227d33abec12eb54101b7e93a3187ddb153a21c5d98aea7396d766b6053efe0a6e11
+  metadata.gz: e16930dc7153e6f296a76ebb8ed08531efa36d0192d89167a832dd2da54e39e1c5840cd0779f6ef60552c3a4508fe9ea657eb3c02a0a740050d32bfb90607c11
+  data.tar.gz: 56d85c860d219502923ce61d75aabced3612cbbeb711e472e1317376220076637a904d177863f933bfd62e82b2ee2c10036d3fd29ed872af106716c650d11b36

data/lib/omniauth/pandadoc/version.rb

@@ -2,6 +2,6 @@
 
 module OmniAuth
   module Pandadoc
-    VERSION = '1.0.0rc2'
+    VERSION = '1.0.0rc3'
   end
 end

data/lib/omniauth/strategies/pandadoc.rb

@@ -5,7 +5,7 @@ module OmniAuth
   module Strategies
     # Main class for Pandadoc OAuth2 strategy.
     class Pandadoc < OmniAuth::Strategies::OAuth2
-      DEFAULT_SCOPE = 'read+write'
+      DEFAULT_SCOPE = 'read+write'.freeze
 
       option :client_options,
              site: 'https://api.pandadoc.com/public/v1',
@@ -14,8 +14,10 @@ module OmniAuth
 
       option :provider_ignores_state, true # this is needed for local testing
 
-      def request_phase
-        redirect client.auth_code.authorize_url({ redirect_uri: callback_url }.merge(options.authorize_params))
+      def authorize_params
+        super.tap do |params|
+          params[:scope] ||= DEFAULT_SCOPE
+        end
       end
     end
   end

data/spec/omniauth/strategies/pandadoc_spec.rb

@@ -105,4 +105,18 @@ describe OmniAuth::Strategies::Pandadoc do
       expect(subject.callback_path).to eq('/auth/foo/callback')
     end
   end
+
+  describe '#authorize_params' do
+    it 'takes a custom scope' do
+      @options = { scope: 'foo' }
+
+      expect(subject.authorize_params[:scope]).to eq 'foo'
+    end
+
+    it 'provides a default scope if none is provided' do
+      @options = {}
+
+      expect(subject.authorize_params[:scope]).to eq described_class::DEFAULT_SCOPE
+    end
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: omniauth-pandadoc
 version: !ruby/object:Gem::Version
-  version: 1.0.0rc2
+  version: 1.0.0rc3
 platform: ruby
 authors:
 - Mike Carey
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-07-20 00:00:00.000000000 Z
+date: 2018-08-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: jwt
@@ -137,7 +137,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: 1.3.1
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.7.6
+rubygems_version: 2.7.7
 signing_key: 
 specification_version: 4
 summary: Pandadoc Strategy for OmniAuth