app.js: prevent mutation of global config (#1233)

Qwant · Nov 24, 2021 · 41ed42d · 41ed42d
1 parent ab1ced9
commit 41ed42d
Showing 1 changed file with 10 additions and 3 deletions.
diff --git a/bin/app.js b/bin/app.js
@@ -125,11 +125,18 @@ function App(config) {
     const userAgent = req.headers['user-agent'];
     const disableMenuRule = config.server.disableBurgerMenu.userAgentRule;
     const { server: _droppedServerConfig, ...appConfig } = config;
+    let localAppConfig = appConfig;
     if (disableMenuRule && userAgent && userAgent.match(disableMenuRule)) {
-      appConfig.burgerMenu.enabled = false;
+      localAppConfig = {
+        ...appConfig,
+        burgerMenu: {
+          ...appConfig.burgerMenu,
+          enabled: false,
+        },
+      };
     }
-    appConfig.testGroupPer = testGroup(config, req).testGroupPer;
-    res.render('index', { config: appConfig });
+    localAppConfig.testGroupPer = testGroup(config, req).testGroupPer;
+    res.render('index', { config: localAppConfig });
   });
 
   if (config.server.acceptPostedLogs) {