fix: allow for content source maps to work with preview.eu.contentful.com (#2545)

Co-authored-by: Bram Kaashoek <[email protected]>

Author: elylucas

lib/utils/validate-params.ts

@@ -64,12 +64,12 @@ export function checkIncludeContentSourceMapsParamIsAllowed(
     )
   }
 
-  const includeContentSourceMapsIsAllowed = host === 'preview.contentful.com'
+  const includeContentSourceMapsIsAllowed = typeof host === 'string' && host.startsWith('preview')
 
   if (includeContentSourceMaps && !includeContentSourceMapsIsAllowed) {
     throw new ValidationError(
       'includeContentSourceMaps',
-      `The 'includeContentSourceMaps' parameter can only be used with the CPA. Please set host to 'preview.contentful.com' to include Content Source Maps.
+      `The 'includeContentSourceMaps' parameter can only be used with the CPA. Please set host to 'preview.contentful.com' or 'preview.eu.contentful.com' to include Content Source Maps.
       `,
     )
   }
@@ -92,7 +92,7 @@ export function checkEnableTimelinePreviewIsAllowed(
   if (isValidConfig && !isValidHost) {
     throw new ValidationError(
       'timelinePreview',
-      `The 'timelinePreview' parameter can only be used with the CPA. Please set host to 'preview.contentful.com' to enable Timeline Preview.
+      `The 'timelinePreview' parameter can only be used with the CPA. Please set host to 'preview.contentful.com' or 'preview.eu.contentful.com' to enable Timeline Preview.
       `,
     )
   }

test/integration/getAsset.test.ts

@@ -40,7 +40,7 @@ describe('getAsset', () => {
   describe('has includeContentSourceMaps enabled', () => {
     test('cdn client', async () => {
       await expect(invalidClient.getAsset(asset)).rejects.toThrow(
-        `The 'includeContentSourceMaps' parameter can only be used with the CPA. Please set host to 'preview.contentful.com' to include Content Source Maps.`,
+        `The 'includeContentSourceMaps' parameter can only be used with the CPA. Please set host to 'preview.contentful.com' or 'preview.eu.contentful.com' to include Content Source Maps.`,
       )
       await expect(invalidClient.getAsset(asset)).rejects.toThrow(ValidationError)
     })

test/integration/getAssets.test.ts

@@ -48,7 +48,7 @@ describe('getAssets', () => {
   describe('has includeContentSourceMaps enabled', () => {
     test('cdn client', async () => {
       await expect(invalidClient.getAssets()).rejects.toThrow(
-        `The 'includeContentSourceMaps' parameter can only be used with the CPA. Please set host to 'preview.contentful.com' to include Content Source Maps.`,
+        `The 'includeContentSourceMaps' parameter can only be used with the CPA. Please set host to 'preview.contentful.com' or 'preview.eu.contentful.com' to include Content Source Maps.`,
       )
       await expect(invalidClient.getAssets()).rejects.toThrow(ValidationError)
     })

test/integration/getEntries.test.ts

@@ -385,7 +385,7 @@ describe('getEntries via client chain modifiers', () => {
   describe('has includeContentSourceMaps enabled', () => {
     test('invalid client', async () => {
       await expect(invalidClient.getEntries()).rejects.toThrow(
-        `The 'includeContentSourceMaps' parameter can only be used with the CPA. Please set host to 'preview.contentful.com' to include Content Source Maps.`,
+        `The 'includeContentSourceMaps' parameter can only be used with the CPA. Please set host to 'preview.contentful.com' or 'preview.eu.contentful.com' to include Content Source Maps.`,
       )
       await expect(invalidClient.getEntries()).rejects.toThrow(ValidationError)
     })

test/integration/getEntry.test.ts

@@ -197,7 +197,7 @@ describe('getEntry via client chain modifiers', () => {
   describe('preview client has includeContentSourceMaps enabled', () => {
     test('invalid client', async () => {
       await expect(invalidClient.getEntry(entryWithResolvableLink)).rejects.toThrow(
-        `The 'includeContentSourceMaps' parameter can only be used with the CPA. Please set host to 'preview.contentful.com' to include Content Source Maps.`,
+        `The 'includeContentSourceMaps' parameter can only be used with the CPA. Please set host to 'preview.contentful.com' or 'preview.eu.contentful.com' to include Content Source Maps.`,
       )
       await expect(invalidClient.getEntry(entryWithResolvableLink)).rejects.toThrow(ValidationError)
     })

test/unit/utils/validate-params.test.ts

@@ -26,6 +26,12 @@ describe('checkIncludeContentSourceMapsParamIsAllowed', () => {
     expect(checkIncludeContentSourceMapsParamIsAllowed('preview.contentful.com', true)).toBe(true)
   })
 
+  it('returns true if includeContentSourceMaps is true and host is preview.eu.contentful.com', () => {
+    expect(checkIncludeContentSourceMapsParamIsAllowed('preview.eu.contentful.com', true)).toBe(
+      true,
+    )
+  })
+
   it('returns false if includeContentSourceMaps is false, regardless of host', () => {
     expect(checkIncludeContentSourceMapsParamIsAllowed('preview.contentful.com', false)).toBe(false)
     expect(checkIncludeContentSourceMapsParamIsAllowed('cdn.contentful.com', false)).toBe(false)