From ad520bb032d5ef04aa1f91efc9f2554fcce11b06 Mon Sep 17 00:00:00 2001
From: David Nguyen <davidngu28@gmail.com>
Date: Thu, 27 Feb 2025 14:08:59 +1100
Subject: [PATCH] fix: remove oauth from embeds

---
 .../embed/embed-authentication-required.tsx     | 13 +++++++------
 apps/remix/app/entry.client.tsx                 |  1 +
 apps/remix/app/root.tsx                         |  2 ++
 apps/remix/package.json                         |  4 ++--
 apps/remix/server/middleware.ts                 |  4 +++-
 package-lock.json                               | 17 +++++++++--------
 packages/lib/package.json                       |  1 +
 7 files changed, 25 insertions(+), 17 deletions(-)

diff --git a/apps/remix/app/components/embed/embed-authentication-required.tsx b/apps/remix/app/components/embed/embed-authentication-required.tsx
index d86256dce..db65e7a2f 100644
--- a/apps/remix/app/components/embed/embed-authentication-required.tsx
+++ b/apps/remix/app/components/embed/embed-authentication-required.tsx
@@ -16,9 +16,9 @@ export type EmbedAuthenticationRequiredProps = {
 export const EmbedAuthenticationRequired = ({
   email,
   returnTo,
-  isGoogleSSOEnabled,
-  isOIDCSSOEnabled,
-  oidcProviderLabel,
+  // isGoogleSSOEnabled,
+  // isOIDCSSOEnabled,
+  // oidcProviderLabel,
 }: EmbedAuthenticationRequiredProps) => {
   return (
     <div className="flex min-h-[100dvh] w-full items-center justify-center">
@@ -35,9 +35,10 @@ export const EmbedAuthenticationRequired = ({
         </Alert>
 
         <SignInForm
-          isGoogleSSOEnabled={isGoogleSSOEnabled}
-          isOIDCSSOEnabled={isOIDCSSOEnabled}
-          oidcProviderLabel={oidcProviderLabel}
+          // Embed currently not supported.
+          // isGoogleSSOEnabled={isGoogleSSOEnabled}
+          // isOIDCSSOEnabled={isOIDCSSOEnabled}
+          // oidcProviderLabel={oidcProviderLabel}
           className="mt-4"
           initialEmail={email}
           returnTo={returnTo}
diff --git a/apps/remix/app/entry.client.tsx b/apps/remix/app/entry.client.tsx
index 6878e9984..4b5769ff5 100644
--- a/apps/remix/app/entry.client.tsx
+++ b/apps/remix/app/entry.client.tsx
@@ -17,6 +17,7 @@ function PosthogInit() {
     if (postHogConfig) {
       posthog.init(postHogConfig.key, {
         api_host: postHogConfig.host,
+        capture_exceptions: true,
       });
     }
   }, []);
diff --git a/apps/remix/app/root.tsx b/apps/remix/app/root.tsx
index 8e72dee2d..759614306 100644
--- a/apps/remix/app/root.tsx
+++ b/apps/remix/app/root.tsx
@@ -1,6 +1,7 @@
 import { useEffect } from 'react';
 
 import Plausible from 'plausible-tracker';
+import posthog from 'posthog-js';
 import {
   Links,
   Meta,
@@ -180,6 +181,7 @@ export function ErrorBoundary({ error }: Route.ErrorBoundaryProps) {
 
   if (errorCode !== 404) {
     console.error('[RootErrorBoundary]', error);
+    posthog.captureException(error);
   }
 
   return <GenericErrorLayout errorCode={errorCode} />;
diff --git a/apps/remix/package.json b/apps/remix/package.json
index f5971dbc2..0bdf674b5 100644
--- a/apps/remix/package.json
+++ b/apps/remix/package.json
@@ -49,8 +49,8 @@
     "luxon": "^3.4.0",
     "papaparse": "^5.4.1",
     "plausible-tracker": "^0.3.9",
-    "posthog-js": "^1.223.3",
-    "posthog-node": "^4.7.0",
+    "posthog-js": "^1.224.0",
+    "posthog-node": "^4.8.1",
     "react": "^18",
     "react-call": "^1.3.0",
     "react-dom": "^18",
diff --git a/apps/remix/server/middleware.ts b/apps/remix/server/middleware.ts
index b7fca6731..5fb194ec1 100644
--- a/apps/remix/server/middleware.ts
+++ b/apps/remix/server/middleware.ts
@@ -49,7 +49,9 @@ export const appMiddleware = async (c: Context, next: Next) => {
   if (pathname.startsWith('/t/')) {
     debug.log('Setting preferred team url cookie');
 
-    setCookie(c, 'preferred-team-url', pathname.split('/')[2]);
+    setCookie(c, 'preferred-team-url', pathname.split('/')[2], {
+      sameSite: 'lax',
+    });
 
     return;
   }
diff --git a/package-lock.json b/package-lock.json
index 1236eeaeb..58e67c573 100644
--- a/package-lock.json
+++ b/package-lock.json
@@ -131,8 +131,8 @@
         "luxon": "^3.4.0",
         "papaparse": "^5.4.1",
         "plausible-tracker": "^0.3.9",
-        "posthog-js": "^1.223.3",
-        "posthog-node": "^4.7.0",
+        "posthog-js": "^1.224.0",
+        "posthog-node": "^4.8.1",
         "react": "^18",
         "react-call": "^1.3.0",
         "react-dom": "^18",
@@ -902,9 +902,9 @@
       }
     },
     "apps/remix/node_modules/posthog-node": {
-      "version": "4.7.0",
-      "resolved": "https://registry.npmjs.org/posthog-node/-/posthog-node-4.7.0.tgz",
-      "integrity": "sha512-RgdUKSW8MfMOkjUa8cYVqWndNjPePNuuxlGbrZC6z1WRBsVc6TdGl8caidmC10RW8mu/BOfmrGbP4cRTo2jARg==",
+      "version": "4.8.1",
+      "resolved": "https://registry.npmjs.org/posthog-node/-/posthog-node-4.8.1.tgz",
+      "integrity": "sha512-ApMEC1+DbctP/88+VhaCl8SRKpIoReibMf7Mb3rxw3yMthr1rKaM4opbHdZJ0buLhwS5zX8B2ckqLjpwpSjRPg==",
       "license": "MIT",
       "dependencies": {
         "axios": "^1.7.4"
@@ -31060,9 +31060,9 @@
       "license": "MIT"
     },
     "node_modules/posthog-js": {
-      "version": "1.223.3",
-      "resolved": "https://registry.npmjs.org/posthog-js/-/posthog-js-1.223.3.tgz",
-      "integrity": "sha512-ZQTc17M21IzkQmECJa2Xjont4tZrvIn252uGT3sTfmahTqZoW4j+kBj4eOJt9SNR6hOheFNkg7MSiI/rA6FaDA==",
+      "version": "1.224.0",
+      "resolved": "https://registry.npmjs.org/posthog-js/-/posthog-js-1.224.0.tgz",
+      "integrity": "sha512-JT1XQQeYs0CKb4lU2vujmeLTDLWc61I5lT7d6oG/H/cnCpXAqBi5rMuCFFeotHeMy3hqJ/Tpu3eAPFE2p5ErHA==",
       "license": "MIT",
       "dependencies": {
         "core-js": "^3.38.1",
@@ -41567,6 +41567,7 @@
         "pdf-lib": "^1.17.1",
         "pg": "^8.11.3",
         "playwright": "1.43.0",
+        "posthog-js": "^1.224.0",
         "react": "^18",
         "remeda": "^2.17.3",
         "sharp": "0.32.6",
diff --git a/packages/lib/package.json b/packages/lib/package.json
index 374a22226..8afe56c85 100644
--- a/packages/lib/package.json
+++ b/packages/lib/package.json
@@ -45,6 +45,7 @@
     "pdf-lib": "^1.17.1",
     "pg": "^8.11.3",
     "playwright": "1.43.0",
+    "posthog-js": "^1.224.0",
     "react": "^18",
     "remeda": "^2.17.3",
     "sharp": "0.32.6",