added site parameter to consumer oauth options

renatosnrg · May 9, 2011 · 90b6641 · 90b6641 · johncant · Aug 3, 2011
1 parent 3b660ce
commit 90b6641
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/lib/linked_in/helpers/authorization.rb b/lib/linked_in/helpers/authorization.rb
@@ -48,6 +48,7 @@ def parse_oauth_options
             :request_token_url => full_oauth_url_for(:request_token, :api_host),
             :access_token_url  => full_oauth_url_for(:access_token,  :api_host),
             :authorize_url     => full_oauth_url_for(:authorize,     :auth_host),
+            :site              => @consumer_options[:site] || @consumer_options[:api_host] || DEFAULT_OAUTH_OPTIONS[:api_host]
           }
         end
 

diff --git a/spec/cases/oauth_spec.rb b/spec/cases/oauth_spec.rb
@@ -13,7 +13,7 @@
       let(:consumer) { client.consumer }
 
       it "should return a configured OAuth consumer" do
-        consumer.site.should == ''
+        consumer.site.should == 'https://api.linkedin.com'
         consumer.request_token_url.should == 'https://api.linkedin.com/uas/oauth/requestToken'
         consumer.access_token_url.should == 'https://api.linkedin.com/uas/oauth/accessToken'
         consumer.authorize_url.should == 'https://www.linkedin.com/uas/oauth/authorize'
@@ -29,6 +29,7 @@
       end
 
       it "should return a configured OAuth consumer" do
+        consumer.site.should == 'https://api.josh.com'
         consumer.request_token_url.should == 'https://api.josh.com/uas/oauth/requestToken'
         consumer.access_token_url.should == 'https://api.josh.com/uas/oauth/accessToken'
         consumer.authorize_url.should == 'https://www.josh.com/uas/oauth/authorize'
@@ -45,6 +46,7 @@
       end
 
       it "should return a configured OAuth consumer" do
+        consumer.site.should == 'https://api.linkedin.com'
         consumer.request_token_url.should == 'https://api.linkedin.com/secure/oauth/requestToken'
         consumer.access_token_url.should == 'https://api.linkedin.com/secure/oauth/accessToken'
         consumer.authorize_url.should == 'https://www.linkedin.com/secure/oauth/authorize'
@@ -61,6 +63,7 @@
       end
 
       it "should return a configured OAuth consumer" do
+        consumer.site.should == 'https://api.linkedin.com'
         consumer.request_token_url.should == 'https://api.josh.com/secure/oauth/requestToken'
         consumer.access_token_url.should == 'https://api.josh.com/secure/oauth/accessToken'
         consumer.authorize_url.should == 'https://www.josh.com/secure/oauth/authorize'
@@ -75,6 +78,7 @@
       end
 
       it "should return a configured OAuth consumer" do
+        consumer.site.should == 'https://api.josh.com'
         consumer.request_token_url.should == 'https://api.josh.com/uas/oauth/requestToken'
         consumer.access_token_url.should == 'https://api.josh.com/uas/oauth/accessToken'
         consumer.authorize_url.should == 'https://api.josh.com/uas/oauth/authorize'