Avoid routing /groups/:groupName/settings as a dataset

Redirect to the group's home page instead, a la …/narratives. Presumably this route will serve a settings page instead in the future.
nextstrain · Feb 15, 2023 · 22e49a3 · 22e49a3
1 parent 1d2eef6
commit 22e49a3
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/src/app.js b/src/app.js
@@ -315,6 +315,12 @@ app.routeAsync("/groups/:groupName")
 app.use("/groups/:groupName/settings",
   endpoints.groups.setGroup(req => req.params.groupName));
 
+/* Avoid matching "settings" as a dataset name; presumably this will eventually
+ * serve a settings page instead.
+ */
+app.routeAsync("/groups/:groupName/settings")
+  .getAsync((req, res) => res.redirect(`/groups/${esc(req.params.groupName)}`));
+
 app.routeAsync("/groups/:groupName/settings/logo")
   .getAsync(getGroupLogo)
   .putAsync(putGroupLogo)