Use cgi.parse_qs instead of urlparse one, import ExpatError properly. Closes gh-78

diff --git a/social_auth/backends/contrib/linkedin.py b/social_auth/backends/contrib/linkedin.py
index 937f75c39631e0b7aaa1d2f11789968072ee0115..af5287acd47501ef44054dd373fe122d27666675 100644 (file)
--- a/social_auth/backends/contrib/linkedin.py
+++ b/social_auth/backends/contrib/linkedin.py
@@ -3,8 +3,9 @@ Linkedin OAuth support
 
 No extra configurations are needed to make this work.
 """
-import urlparse
+from cgi import parse_qs
 from xml.etree import ElementTree
+from xml.parsers.expat import ExpatError
 
 from social_auth.backends import ConsumerBasedOAuth, OAuthBackend
 
@@ -51,10 +52,9 @@ class LinkedinAuth(ConsumerBasedOAuth):
             data = _xml_to_dict(xml)
             url = data['site-standard-profile-request']['url']
             url = url.replace('&', '&')
-            data['id'] = urlparse.parse_qs(url)['key'][0]
-
+            data['id'] = parse_qs(url)['key'][0]
             return data
-        except (xml.parsers.expat.ExpatError, KeyError, IndexError):
+        except (ExpatError, KeyError, IndexError):
             return None
 
     @classmethod