Merge pull request #3 from bouvet-apps/bugfix/site-not-mounted-on-dom…

…ain-root Fix bug with matomo only working on site mounted on domain root
bouvet-apps · Feb 24, 2023 · 2b50a55 · 2b50a55
2 parents f43fdc3 + 85f41e5
commit 2b50a55
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/gradle.properties b/gradle.properties
@@ -8,4 +8,4 @@ displayName = Matomo Analytics App
 vendor = Bouvet Norge AS
 vendorUrl = https://www.bouvet.no
 xpVersion = 7.5.0
-version = 1.3.0
+version = 1.3.1
diff --git a/src/main/resources/site/processors/matomo-report.js b/src/main/resources/site/processors/matomo-report.js
@@ -62,7 +62,13 @@ exports.responseProcessor = function (req, res) {
     hash = hashCode(req.cookies["no-bouvet-app-matomo_disabled"] + ""); // Create a unique hash if user has consented to tracking.
   }
 
-  res.pageContributions.headEnd.push("<script async defer src=\"matomo.js?" + hash + "\"></script>");
+  let siteRootPath = portalLib.pageUrl({ id: portalLib.getSite()._id });
+  // Site vhost is mounted on domain root, e.g. www.example.com
+  if (siteRootPath === "/") {
+    siteRootPath = "";
+  }
+
+  res.pageContributions.headEnd.push("<script async defer src=\"" + siteRootPath + "/matomo.js?" + hash + "\"></script>");
   if (matomoTagManagerContainerId) {
     res.pageContributions.headEnd.push("<script async defer src=\"" + matomoJavaScriptUrl + "/container_" + matomoTagManagerContainerId + ".js\"></script>");
   }