lmno: Be forgiving in game ID values received from the user

Most significantly, we now accept a lowercase version of the ID.

But also, game ID values are intentionally chosen not to have
ambiguous character in them, (such as "M"/"N" or "S"/"F"), which might
sound similar when someone says a game ID aloud. So on the input side
we accept either form and canonicalize to the only letter which can
possibly exist in a game ID.

diff --git a/lmno.js b/lmno.js
index f79da3afd653cf1d7b89219aada1546d97aff036..64a106bf9a492f0475c5e6a9800a306b0c5bfcdb 100644 (file)
--- a/lmno.js
+++ b/lmno.js
@@ -46,6 +46,24 @@ LMNO.letters = "BCDFGHJKLMQRTVWXYZ";
 
 const lmno = new LMNO();
 
+/* Force a game ID into a canonical form as described above. */
+function lmno_canonize(id) {
+  /* Capitalize */
+  id = id.toUpperCase();
+
+  /* Replace unused letters with nearest phonetic match. */
+  id = id.replace(/N/g, 'M');
+  id = id.replace(/P/g, 'B');
+  id = id.replace(/S/g, 'F');
+
+  /* Replace unused numbers nearest visual match. */
+  id = id.replace(/0/g, 'O');
+  id = id.replace(/1/g, 'I');
+  id = id.replace(/5/g, 'S');
+
+  return id;
+}
+
 app.post('/new/:game_engine', (request, response) =>  {
   const game_engine = request.params.game_engine;
   const game_id = lmno.create_game(game_engine);
@@ -60,6 +78,13 @@ app.post('/new/:game_engine', (request, response) =>  {
  */
 app.get('/[a-zA-Z0-9]{4}', (request, response) => {
   const game_id = request.path.replace(/\//g, "");
+  const canon_id = lmno_canonize(game_id);
+
+  /* Redirect user to page with the canonical ID in it. */
+  if (game_id !== canon_id) {
+    response.redirect(301, `/${canon_id}/`);
+    return;
+  }
 
   const game = lmno.ids[game_id];
   if (game === undefined) {
@@ -71,7 +96,18 @@ app.get('/[a-zA-Z0-9]{4}', (request, response) => {
 
 /* LMNO middleware to lookup the game. */
 app.use('/empires/:game_id([a-zA-Z0-9]{4})', (request, response, next) => {
-  request.game = lmno.ids[request.params.game_id].game;
+  const game_id = request.params.game_id;
+  const canon_id = lmno_canonize(game_id);
+
+  /* Redirect user to page with the canonical ID in it. */
+  if (game_id !== canon_id) {
+    const new_url = request.originalUrl.replace("/empires/" + game_id,
+                                                "/empires/" + canon_id);
+    response.redirect(301, new_url);
+    return;
+  }
+
+  request.game = lmno.ids[game_id].game;
   if (request.game === undefined) {
     response.sendStatus(404);
     return;