Capture data on whether the incoming URL has a slash on it.

Author: ericholscher

readthedocs/proxito/tests/test_redirects.py

@@ -40,6 +40,15 @@ def test_subproject_root_url_no_slash(self):
             r['Location'], 'https://project.dev.readthedocs.io/projects/subproject/en/latest/',
         )
 
+    def test_single_version_subproject_root_url_no_slash(self):
+        self.subproject.single_version = True
+        self.subproject.save()
+        r = self.client.get('/projects/subproject', HTTP_HOST='project.dev.readthedocs.io')
+        self.assertEqual(r.status_code, 302)
+        self.assertEqual(
+            r['Location'], 'https://project.dev.readthedocs.io/projects/subproject/',
+        )
+
     def test_root_redirect_with_query_params(self):
         r = self.client.get('/?foo=bar', HTTP_HOST='project.dev.readthedocs.io')
         self.assertEqual(r.status_code, 302)

readthedocs/proxito/urls.py

@@ -147,9 +147,9 @@
     # (Sub)project single version
     url(
         (
-            # the /? at the end of this regex is for ``/projects/subproject``
+            # subproject_slash variable at the end of this regex is for ``/projects/subproject``
             # so that it will get captured here and redirect properly.
-            r'^(?:projects/(?P<subproject_slug>{project_slug})/?)?'
+            r'^(?:projects/(?P<subproject_slug>{project_slug})(?P<subproject_slash>/?))?'
             r'(?P<filename>{filename_slug})$'.format(**pattern_opts)
         ),
         ServeDocs.as_view(),

readthedocs/proxito/views/serve.py

@@ -55,11 +55,17 @@ def get(self,
             request,
             project_slug=None,
             subproject_slug=None,
+            subproject_slash=None,
             lang_slug=None,
             version_slug=None,
             filename='',
     ):  # noqa
-        """Take the incoming parsed URL's and figure out what file to serve."""
+        """
+        Take the incoming parsed URL's and figure out what file to serve.
+
+        ``subproject_slash`` is used to determine if the subproject URL has a slash,
+        so that we can decide if we need to serve docs or add a /.
+        """
 
         version_slug = self.get_version_from_host(request, version_slug)
         final_project, lang_slug, version_slug, filename = _get_project_data_from_request(  # noqa
@@ -87,6 +93,14 @@ def get(self,
         ]):
             return self.system_redirect(request, final_project, lang_slug, version_slug, filename)
 
+        # Handle `/projects/subproject` URL redirection
+        if all([
+                final_project.single_version,
+                filename == '',
+                not subproject_slash,
+        ]):
+            return self.system_redirect(request, final_project, lang_slug, version_slug, filename)
+
         if all([
                 (lang_slug is None or version_slug is None),
                 not final_project.single_version,