]> git.cworth.org Git - empires-server/commitdiff
Add 'trust proxy' option since we trust our proxy
authorCarl Worth <cworth@cworth.org>
Sat, 6 Jun 2020 16:34:29 +0000 (09:34 -0700)
committerCarl Worth <cworth@cworth.org>
Sat, 6 Jun 2020 16:34:29 +0000 (09:34 -0700)
This fixes the URL that the server gives to clients in the game-info
event so that it will be a working URL (https://lmno.games/WXYZ)
instead of a URL from the node point of view (http://localhost/WXYZ)
which won't do any good for users to share.

lmno.js

diff --git a/lmno.js b/lmno.js
index 688c7a74e9800e51fde3a1375dcc28239e0862f3..d01b3e9478543b20c9a51196677a30a02c30edef 100644 (file)
--- a/lmno.js
+++ b/lmno.js
@@ -32,6 +32,15 @@ The "node lmno-passwd.js" command can help generate password hashes.`);
 }
 
 const app = express();
+
+/* This 'trust proxy' option, (and, really? a space in an option
+ * name?!)  means that express will grab hostname and IP values from
+ * the X-Forwarded-* header fields. We need that so that our games
+ * will display a proper hostname of https://lmno.games/WXYZ instead
+ * of http://localhost/QFBL which will obviously not be a helpful
+ * thing to share around.
+ */
+app.set('trust proxy', true);
 app.use(cors());
 app.use(body_parser.urlencoded({ extended: false }));
 app.use(body_parser.json());