fix: ensure plausible always gets data

Kukei-eu · Dec 17, 2023 · 1365b78 · 1365b78
1 parent f000b18
commit 1365b78
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 7 deletions.
diff --git a/functions/index.js b/functions/index.js
@@ -46,7 +46,8 @@ export const onRequestGet = async (context) => {
 
 	const hasResults = results.length > 0;
 
-	emitPageView(context, {
+	// without await it might get killed before sending by cloudflare
+	await emitPageView(context, {
 		hasResults,
 	});
 

diff --git a/lib/mongo.js b/lib/mongo.js
@@ -10,6 +10,8 @@ const callMongo = async (envs, action, data) => {
 		body: JSON.stringify(data)
 	});
 
+	console.log(request);
+
 	const response = await request.json();
 
 	return response;

diff --git a/lib/plausible.js b/lib/plausible.js
@@ -39,13 +39,15 @@ const emitEvent = async (context, name, props = {}) => {
 };
 /**
  * Emits PageView to Plausible.io
- * Note: this is filly GDPR/Telecom compliant as we anonymize the IP address.
+ * Note: this is fully GDPR/Telecom compliant as we anonymize the IP address.
  * Should never throw. Should be fire and forget.
  * @param context
+ * @param {Object} props
  */
-export const emitPageView = (context, props) => {
-	emitEvent(context, 'pageview', props)
-		.catch((error) => {
-			console.error('Plausible failed', error);
-		});
+export const emitPageView = async (context, props) => {
+	try {
+		await emitEvent(context, 'pageview', props);
+	} catch (error) {
+		console.error('Plausible failed', error);
+	}
 };