omniauth-osso 0.1.8 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 56e18431f00e10031a36b55c2ebc0fbb472b27fb98cc794bd0925a80bd6a85f4
-  data.tar.gz: 9cf0b3999072a6cceb1ff764cb3c55d5f0c16d0dd122f24c1b8fee6c92eb29d6
+  metadata.gz: 257baf2f9360500ab7c89d965a6591ed5f19fae096435b409c14ab5398d35e1a
+  data.tar.gz: 1ce66747bf46e62db83a24b73c312ae2c32fd0455a41784fb8654de14e1dd372
 SHA512:
-  metadata.gz: 26ac30a6016727d1a97bcd3c2b6a49911da1b044487fb626b111586552e9ccf4cd8dd30b2866e3c4bc84ceb7218fb6200a745ffcc4e34ad906105029a24f16a6
-  data.tar.gz: 92dc68770cc6445875a73375b02f5bc56704630ba735635d97b686476cdf80ad1cbb87a77cbcb030b4cb5c1a8e65a7b0bdc854de6bb3723b3c3fd2decd44d7aa
+  metadata.gz: 76e37f338722da1d8742df1c744e4e21303bb5796e7cba8b9744eb028dfc4b33a5570eced3ecb63f8029252bbb2322af612da954b7fea4456a063ef3f29fafed
+  data.tar.gz: 48f549fce5708546952078bd4b8180edc1d76685d24109c08d3a15d30fb0df7ea92f8d057f8beeb4cf96b77a22511fe63e7a65e1fd77644d0fbffaddf222d497

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    omniauth-osso (0.1.8.pre)
+    omniauth-osso (0.2.0)
       omniauth (~> 2.0.1)
       omniauth-oauth2 (>= 1.6, < 1.8)
 

data/lib/omniauth-osso/version.rb

@@ -2,6 +2,6 @@
 
 module OmniAuth
   module Osso
-    VERSION = '0.1.8'
+    VERSION = '0.2.0'
   end
 end

data/lib/omniauth/strategies/osso.rb

@@ -88,10 +88,13 @@ module OmniAuth
       def user_param
         return @user_param if defined?(@user_param)
 
-        @user_param = {
-          domain: request.params['domain'],
-          email: request.params['email']
-        }.compact
+        params = request.params.reject { |_k, v| v.empty? }
+
+        @user_param = {}
+        @user_param = { domain: request.params['domain'] } if params['domain']
+        @user_param = { email: request.params['email'] } if params['email']
+
+        @user_param
       end
     end
   end

data/spec/omniauth/strategies/osso_spec.rb

@@ -70,13 +70,48 @@ describe OmniAuth::Strategies::Osso do
       expect(instance.request_params[:domain]).to eq('example.com')
     end
 
-    it 'includes email when an email address is passed as an authorize option' do
+    it 'includes email passed as a request param' do
       instance = subject.new('abc', 'def')
       instance.env = {}
       allow(instance).to receive(:request) do
         double('Request', params: { 'email' => 'user@example.com' }, scheme: 'https', url: url)
       end
+
+      expect(instance.request_params[:email]).to eq('user@example.com')
+    end
+
+    it 'only includes email as a request param when both keys are provided' do
+      instance = subject.new('abc', 'def')
+      instance.env = {}
+      allow(instance).to receive(:request) do
+        double('Request', params: { 'email' => 'user@example.com', 'domain' => 'example.com' }, scheme: 'https',
+                          url: url)
+      end
+
       expect(instance.request_params[:email]).to eq('user@example.com')
+      expect(instance.request_params.keys).to_not include(:domain)
+    end
+
+    it 'domain as a request param when email key is a blank string' do
+      instance = subject.new('abc', 'def')
+      instance.env = {}
+      allow(instance).to receive(:request) do
+        double('Request', params: { 'email' => '', 'domain' => 'example.com' }, scheme: 'https',
+                          url: url)
+      end
+
+      expect(instance.request_params[:domain]).to eq('example.com')
+      expect(instance.request_params.keys).to_not include(:email)
+    end
+
+    it 'only includes redirect_uri as a request param if neither email or domain are provided' do
+      instance = subject.new('abc', 'def')
+      instance.env = {}
+      allow(instance).to receive(:request) do
+        double('Request', params: {}, scheme: 'https', url: url)
+      end
+
+      expect(instance.request_params.keys).to eq([:redirect_uri])
     end
   end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: omniauth-osso
 version: !ruby/object:Gem::Version
-  version: 0.1.8
+  version: 0.2.0
 platform: ruby
 authors:
 - Sam Bauch
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-01-25 00:00:00.000000000 Z
+date: 2021-01-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: omniauth