chore(data-warehouse): Dont allow S3 protocol for linking data source (…

…#24602)
PostHog · Aug 27, 2024 · af509c8 · af509c8
1 parent b8dd904
commit af509c8
Showing 1 changed file with 7 additions and 0 deletions.
diff --git a/frontend/src/scenes/data-warehouse/new/dataWarehouseTableLogic.tsx b/frontend/src/scenes/data-warehouse/new/dataWarehouseTableLogic.tsx
@@ -92,6 +92,13 @@ export const dataWarehouseTableLogic = kea<dataWarehouseTableLogicType>([
         table: {
             defaults: { ...NEW_WAREHOUSE_TABLE } as DataWarehouseTable,
             errors: ({ name, url_pattern, credential, format }) => {
+                if (url_pattern?.startsWith('s3://')) {
+                    return {
+                        url_pattern:
+                            'Please use the https version of your bucket url e.g. https://your-org.s3.amazonaws.com/airbyte/stripe/invoices/*.pqt',
+                    }
+                }
+
                 return {
                     name: !name && 'Please enter a name.',
                     url_pattern: !url_pattern && 'Please enter a url pattern.',