Merge pull request #27 from spraints/html_url

Set info.urls['GitHub'] from the user info
omniauth · Feb 13, 2015 · e0304e7 · e0304e7
2 parents 48d718c + dcc846a
commit e0304e7
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/lib/omniauth/strategies/github.rb b/lib/omniauth/strategies/github.rb
@@ -32,7 +32,7 @@ def authorize_params
           'name' => raw_info['name'],
           'image' => raw_info['avatar_url'],
           'urls' => {
-            'GitHub' => "https://github.com/#{raw_info['login']}",
+            'GitHub' => raw_info['html_url'],
             'Blog' => raw_info['blog'],
           },
         }

diff --git a/spec/omniauth/strategies/github_spec.rb b/spec/omniauth/strategies/github_spec.rb
@@ -132,4 +132,11 @@
     end
   end
 
+  context '#info.urls' do
+    it 'should use html_url from raw_info' do
+      subject.stub(:raw_info).and_return({ 'login' => 'me', 'html_url' => 'http://enterprise/me' })
+      subject.info['urls']['GitHub'].should == 'http://enterprise/me'
+    end
+  end
+
 end