From 3df7a9232a150ec1dc2cb8da3835e8083e011e55 Mon Sep 17 00:00:00 2001 From: Mike McNeil Date: Fri, 31 May 2024 14:31:21 -0500 Subject: [PATCH] Website: Remove utm parameter stripping for now (#19425) --- website/api/hooks/custom/index.js | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/website/api/hooks/custom/index.js b/website/api/hooks/custom/index.js index c686884144..439fe5ed09 100644 --- a/website/api/hooks/custom/index.js +++ b/website/api/hooks/custom/index.js @@ -154,7 +154,12 @@ will be disabled and/or hidden in the UI. // https://fleetdm.com/device-management?utm_content=mdm if (['clear','eo-security', 'eo-it', 'mdm', 'vm'].includes(req.param('utm_content'))) { req.session.primaryBuyingSituation = req.param('utm_content') === 'clear' ? undefined : req.param('utm_content'); - return res.redirect(req.path);// « auto-redirect without querystring to make it prettier in the URL bar. + // FUTURE: reimplement the following (auto-redirect without querystring to make it prettier in the URL bar), but do it in the client-side JS + // using whatever that poppushstateblah thing is that makes it so you can change the URL bar from the browser-side code without screwing up + // the history stack (i.e. back button) + // ``` + // return res.redirect(req.path); + // ``` }//fi if (req.method === 'GET' || req.method === 'HEAD') {