Expect iframe postback data to be a string from POST parameter toopher_iframe_data

Author: Grace Yim

test/test_toopher_iframe.py

@@ -44,7 +44,7 @@ def _get_auth_request_postback_data_as_dict(self):
 
     def _get_urlencoded_auth_request_postback_data(self, auth_request_data = None):
         data = auth_request_data if auth_request_data else self._get_auth_request_postback_data_as_dict()
-        return {'toopher_iframe_data': urllib.urlencode(data)}
+        return urllib.urlencode(data)
 
     def _get_pairing_postback_data_as_dict(self):
         return {
@@ -62,7 +62,7 @@ def _get_pairing_postback_data_as_dict(self):
 
     def _get_urlencoded_pairing_postback_data(self, pairing_data=None):
         data = pairing_data if pairing_data else self._get_pairing_postback_data_as_dict()
-        return {'toopher_iframe_data': urllib.urlencode(data)}
+        return urllib.urlencode(data)
 
     def _get_user_postback_data_as_dict(self):
         return {
@@ -77,7 +77,7 @@ def _get_user_postback_data_as_dict(self):
 
     def _get_urlencoded_user_postback_data(self, user_data=None):
         data = user_data if user_data else self._get_user_postback_data_as_dict()
-        return {'toopher_iframe_data': urllib.urlencode(data)}
+        return urllib.urlencode(data)
 
     def test_process_postback_good_signature_returns_authentication_request(self):
         auth_data = self._get_auth_request_postback_data_as_dict()

toopher/toopher_iframe.py

@@ -55,8 +55,8 @@ def get_user_management_url(self, username, reset_email='', **kwargs):
 
         return self._get_oauth_signed_url(self.base_uri + '/web/manage_user', params, ttl)
 
-    def process_postback(self, urlencoded_form_data, request_token='', **kwargs):
-        toopher_data = self._urldecode_iframe_data(urlencoded_form_data)
+    def process_postback(self, urlencoded_toopher_data, request_token='', **kwargs):
+        toopher_data = self._urldecode_iframe_data(urlencoded_toopher_data)
 
         if 'error_code' in toopher_data:
             error_message = toopher_data['error_message']
@@ -91,7 +91,7 @@ def is_authentication_granted(self, data, request_token='', **kwargs):
             return False
 
     def _urldecode_iframe_data(self, data):
-        data_dict = urlparse.parse_qs(data['toopher_iframe_data'])
+        data_dict = urlparse.parse_qs(data)
         return dict((k,v[0]) for (k,v) in data_dict.items())
 
     def _validate_data(self, data, request_token, kwargs):