From: Matías Aguirre Date: Sat, 28 May 2011 00:00:11 +0000 (-0300) Subject: Use cgi.parse_qs instead of urlparse one, import ExpatError properly. Closes gh-78 X-Git-Url: https://git.parisson.com/?a=commitdiff_plain;h=b8579fa317ba4a44f661ab3f5a33506162524af5;p=django-social-auth.git 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 937f75c..af5287a 100644 --- 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