tea/browsertrix
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

analytics path in dev server (#2231)

Browse Source 
quick follow-up to #2226:
- rename script _plausible.js -> analytics.js to be more generic
- devserver: serve empty /analytics.js from webpack dev server

---------

Co-authored-by: emma <hi@emma.cafe>
This commit is contained in:
Ilya Kreymer 2024-12-12 12:06:03 -08:00 committed by GitHub
parent db39333ef4
commit 3e6788b2c1
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
3 changed files with 11 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
frontend/00-browsertrix-nginx-init.sh
View File

@ -12,13 +12,13 @@ else
sed -i "s/\$LOCAL_BUCKET/$LOCAL_BUCKET/g" /etc/nginx/includes/minio.conf
fi
# Plausible analytics
# Add analytics script, if provided
if [ -z "$INJECT_ANALYTICS" ]; then
echo "analytics disabled, injecting blank script"
echo "" > /usr/share/nginx/html/_plausible.js
echo "" > /usr/share/nginx/html/extra.js
else
echo "analytics enabled, injecting script"
echo "$INJECT_ANALYTICS" > /usr/share/nginx/html/_plausible.js
echo "$INJECT_ANALYTICS" > /usr/share/nginx/html/extra.js
fi
mkdir -p /etc/nginx/resolvers/

2
frontend/src/index.ejs
View File

@ -18,7 +18,7 @@
<link rel="icon" href="/favicon.svg" type="image/svg+xml" />
<link rel="apple-touch-icon" href="/apple-touch-icon.png" />
<link rel="manifest" href="/manifest.webmanifest" />
<script src="/_plausible.js"></script>
<script src="/extra.js"></script>
</head>
<body>
<script>

7
frontend/webpack.dev.js
View File

@ -75,6 +75,13 @@ module.exports = [
res.set("Content-Type", "application/javascript");
res.status(404).send(`{"error": "placeholder_for_replay"}`);
});
// serve empty analytics script
server.app?.get("/extra.js", (req, res) => {
res.set("Content-Type", "application/javascript");
res.status(200).send("");
});
return middlewares;
},
port: 9870,