Handle more parenthesis in broken URL redirects

TurboWarp · Jun 8, 2024 · e4f0e0f · e4f0e0f
1 parent 344edbd
commit e4f0e0f
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/src/server.js b/src/server.js
@@ -181,7 +181,7 @@ app.use((req, res, next) => {
       // /projects/123 to /123 and /https://scratch.mit.edu/projects/123 to /123
       path.match(/^\/(?:https:\/\/scratch\.mit\.edu\/)?projects\/(\d+)\/?(?:(editor|fullscreen|embed)\/?)?$/) ||
       // /123) to /123
-      path.match(/^\/(\d+)\/?(?:(editor|fullscreen|embed)\/?)?\)$/)
+      path.match(/^\/(\d+)\/?(?:(editor|fullscreen|embed)\/?)?\)+$/)
     );
     if (projectMatch) {
       const search = url.parse(req.url).search;

diff --git a/test/rewrite-broken.test.js b/test/rewrite-broken.test.js
@@ -67,3 +67,9 @@ it('redirects /id/editor/) to /id', () => {
     .set('Host', 'localhost')
     .expect('Location', '/104/editor');
 });
+
+it('redirects /id)))))))))))))))))))))))))))))))))))))))))))))))))))))))))))))))))) to /id', () => {
+  return request.get('/104))))))))))))))))))))))))))))))))))))))))))))))))))))))))))))))))))')
+    .set('Host', 'localhost')
+    .expect('Location', '/104');
+});