redirect URI clean up moved to utils module

oidc_provider/lib/endpoints/authorize.py

@@ -25,6 +25,7 @@ from oidc_provider.models import (
     UserConsent,
 )
 from oidc_provider import settings
+from oidc_provider.lib.utils.common import cleanup_url_from_query_string
 
 
 logger = logging.getLogger(__name__)
@@ -85,8 +86,7 @@ class AuthorizeEndpoint(object):
         if self.is_authentication and not self.params['redirect_uri']:
             logger.debug('[Authorize] Missing redirect uri.')
             raise RedirectUriError()
-        clean_redirect_uri = urlsplit(self.params['redirect_uri'])
+        clean_redirect_uri = cleanup_url_from_query_string(self.params['redirect_uri'])
-        clean_redirect_uri = urlunsplit(clean_redirect_uri._replace(query=''))
         if not (clean_redirect_uri in self.client.redirect_uris):
             logger.debug('[Authorize] Invalid redirect uri: %s', self.params['redirect_uri'])
             raise RedirectUriError()

oidc_provider/lib/endpoints/token.py

@@ -2,6 +2,9 @@ from base64 import b64decode, urlsafe_b64encode
 import hashlib
 import logging
 import re
+
+from oidc_provider.lib.utils.common import cleanup_url_from_query_string
+
 try:
     from urllib.parse import unquote
 except ImportError:
@@ -86,7 +89,8 @@ class TokenEndpoint(object):
                 raise TokenError('invalid_client')
 
         if self.params['grant_type'] == 'authorization_code':
-            if not (self.params['redirect_uri'] in self.client.redirect_uris):
+            clean_redirect_uri = cleanup_url_from_query_string(self.params['redirect_uri'])
+            if not (clean_redirect_uri in self.client.redirect_uris):
                 logger.debug('[Token] Invalid redirect uri: %s', self.params['redirect_uri'])
                 raise TokenError('invalid_client')
 

oidc_provider/lib/utils/common.py

@@ -3,6 +3,26 @@ from django.http import HttpResponse
 
 from oidc_provider import settings
 
+try:
+    from urlparse import urlsplit, urlunsplit
+except ImportError:
+    from urllib.parse import urlsplit, urlunsplit
+
+
+def cleanup_url_from_query_string(uri):
+    """
+    Function used to clean up the uri from any query string, used i.e. by endpoints to validate redirect_uri
+
+    :param uri: URI to clean from query string
+    :type uri: str
+    :return: cleaned URI without query string
+    """
+
+    clean_uri = urlsplit(uri)
+    # noinspection PyProtectedMember
+    clean_uri = urlunsplit(clean_uri._replace(query=''))
+    return clean_uri
+
 
 def redirect(uri):
     """

oidc_provider/tests/test_token_endpoint.py

@@ -213,6 +213,31 @@ class TokenTestCase(TestCase):
         response = self._post_request(post_data)
         self.assertIn('invalid_grant', response.content.decode('utf-8'))
 
+    def test_client_redirect_url(self):
+        """
+        Validate that client redirect URIs with query strings match registered
+        URIs, and that unregistered URIs are rejected.
+
+        source: https://github.com/jerrykan/django-oidc-provider/blob/2f54e537666c689dd8448f8bbc6a3a0244b01a97/oidc_provider/tests/test_token_endpoint.py
+        """
+        SIGKEYS = self._get_keys()
+        code = self._create_code()
+        post_data = self._auth_code_post_data(code=code.code)
+
+        # Unregistered URI
+        post_data['redirect_uri'] = 'http://invalid.example.org'
+
+        response = self._post_request(post_data)
+
+        self.assertIn('invalid_client', response.content.decode('utf-8')),
+
+        # Registered URI contained a query string
+        post_data['redirect_uri'] = 'http://example.com/?client=OidcClient'
+
+        response = self._post_request(post_data)
+
+        self.assertNotIn('invalid_client', response.content.decode('utf-8')),
+
     def test_request_methods(self):
         """
         Client sends an HTTP POST request to the Token Endpoint. Other request