omniauth-linkedin-jsapi 0.1.1 → 0.1.2

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    ZTJhNTcwZTE1ZmJiNzViZGQ4ODcyNGYyNjUxMDAyZTc0MjFiNzM5NQ==
+    ZWY3MmU2ZTEzOTAyOTEzMzAzYjM5NDU2YjhkYTU5YmY2NDMyY2FkNw==
   data.tar.gz: !binary |-
-    MDQwMzg0MjI1NWExMmUzOWMwZGY0NzM2ZDZjMGYxN2ZjNmYwNDk4NQ==
+    MjBjN2JhNTUzZDI2Y2IzMjFjZGY0NDczZjFhN2RjNDkyYTk0YzhjNg==
 SHA512:
   metadata.gz: !binary |-
-    OGExYzQ3ZGQzMmY2YjNlYjgxM2Y2ZjdjZjVmYzAwYThkMDFhM2RjZDJkY2Mz
-    ZGRiYTk5NWFmMDRmZmQzZjNkNTRlMjBjMDFjNzFlZmFiMjExMDRmMDA0ZjAz
-    NDBjZWM5ZjQ3NjA2OTMzY2Q5ZjY5MDg4ZjRhN2I4ZDM5ODdhNWQ=
+    ODgzZmZlNGZmZWM2OGQ3NWE3YWQxMjNjNDdjNzVhMzNkMmI2MzljMTYyNDJm
+    ODMxZjc5MmRlMjJiMmE2ZDhlYjdjM2E0M2U2ZjU1OWI4NGQ2YzdjNzQwNzU2
+    MTRmYTk5ODJkZDQ1MmQ4YTNjMDY3MzlhZmQ3Y2QxMzU3ODE0MTg=
   data.tar.gz: !binary |-
-    M2RjMmY1YTU2ODE4ZTljNjlkNjBiZTc0MDAxYTJjOTkxOTk1NDE1M2NkMzJl
-    MmE2YjVhNjFhNTFlMWJlZDIyNDFlMzdlZWM0MDIyZjUzN2Y1NDQ0MmI2MGMw
-    YjdkZDAxNTJmMzY4ZDAyZDk4NTJhNDdjMjgxY2JhMzRiY2IyNzE=
+    NzZjYmFhMzRlZDdkN2RkOGJiNTZmNTlmNDA3MTMzNzNkNmRmODU0YjNhZmYz
+    NGVlOGYzOTg2YjA1ZmE1ZDM2N2MwNjdiNTY5YzlmOGY4ZGNkMzg3Yjg0Y2Nj
+    MWI2OGM2NzEyZjA5NWFkYmI0MWQzYWFkODU4ODk4ZWQ2MmM5MTE=

data/lib/omniauth-linkedin-jsapi/version.rb

@@ -1,5 +1,5 @@
 module OmniAuth
   module LinkedInJsapi
-    VERSION = "0.1.1"
+    VERSION = "0.1.2"
   end
 end

data/lib/omniauth/strategies/linkedin.rb

@@ -12,6 +12,7 @@ module OmniAuth
       include OmniAuth::Strategy
       class NoSecureCookieError < StandardError; end
       class InvalidSecureCookieError < StandardError; end
+      class LinkedinServerError < StandardError; end
 
       args [:api_key, :secret_key]
 
@@ -60,7 +61,23 @@ module OmniAuth
       end
 
       def raw_info
-        @raw_info ||= MultiJson.decode(access_token.get("/v1/people/~:(#{options.fields.join(',')})?format=json").body)
+        @raw_info ||= lambda do
+          # Add retry logic to user info fetching because sometimes Linkedin gets back with the following data:
+          # raw_info: {
+          #   errorCode: 0,
+          #   message: "Could not find person based on: ~",
+          #   requestId: "***************",
+          #   status: 404,
+          #   timestamp: 1393490622830.0
+          # }
+          try_count = 0
+          begin
+            raw_info = MultiJson.decode(access_token.get("/v1/people/~:(#{options.fields.join(',')})?format=json").body)
+            try_count += 1
+          end while raw_info['errorCode'].present? and try_count < 5
+          raise LinkedinServerError, raw_info['message'] if raw_info['errorCode'].present?
+          raw_info
+        end.call
       end
 
       attr_accessor :access_token
@@ -86,6 +103,12 @@ module OmniAuth
         else
           raise NoSecureCookieError, 'must pass a `linkedin_oauth_XXX` cookie'
         end
+      rescue NoSecureCookieError => e
+        fail!(:invalid_credentials, e)
+      rescue InvalidSecureCookieError => e
+        fail!(:invalid_credentials, e)
+      rescue LinkedinServerError => e
+        fail!(:invalid_response, e)
       rescue ::Timeout::Error => e
         fail!(:timeout, e)
       rescue ::Net::HTTPFatalError, ::OpenSSL::SSL::SSLError => e

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: omniauth-linkedin-jsapi
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
 platform: ruby
 authors:
 - Daniel Gong
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-12-23 00:00:00.000000000 Z
+date: 2014-02-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: omniauth