Merge pull request #225 from fingerprintjs/bugfix/INTER-726-invalid-host

[INTER-726] Fix invalid host header

e2e/infra/frontdoor.ts

@@ -10,6 +10,7 @@ import {
 import { KnownSessionAffinityEnabledState } from '@azure/arm-frontdoor'
 import invariant from 'tiny-invariant'
 import { ExponentialBackoff, handleAll, retry } from 'cockatiel'
+import { wait } from '../../shared/wait'
 
 export interface ProvisionFrontDoorParams {
   resourceGroup: string
@@ -155,6 +156,9 @@ export async function provisionFrontDoor({
   }
 }
 
+// 10 minutes
+const ADDITIONAL_WAIT = 600_000
+
 async function waitForFrontDoor(url: string) {
   console.info('Waiting for front door to be ready...', url)
 
@@ -166,7 +170,7 @@ async function waitForFrontDoor(url: string) {
     maxAttempts: 40,
   })
 
-  return policy.execute(async ({ attempt }) => {
+  await policy.execute(async ({ attempt }) => {
     if (attempt > 1) {
       console.info(`Attempt ${attempt}...`)
     }
@@ -181,7 +185,12 @@ async function waitForFrontDoor(url: string) {
     if (text.includes('Page not found')) {
       throw new Error('Page not found')
     }
-
-    console.log('Frontdoor is ready!')
   })
+
+  console.info(
+    `Frontdoor is ready! Waiting additional ${ADDITIONAL_WAIT / 1000} seconds to make sure it is fully operational...`
+  )
+
+  // Wait additional ADDITIONAL_WAIT ms, to make sure that frontdoor is fully ready. Otherwise it might cause flaky tests.
+  await wait(ADDITIONAL_WAIT)
 }

e2e/scripts/mockTests.ts

@@ -1,8 +1,9 @@
 import { execSync } from 'child_process'
 import { readTestInfo } from '../shared/testInfo'
 import pkg from '../../package.json'
+import { ExponentialBackoff, handleAll, retry } from 'cockatiel'
 
-async function main() {
+async function doMockTests() {
   let hasError = false
   const testInfo = readTestInfo()
 
@@ -15,7 +16,7 @@ async function main() {
   for (const info of testInfo.tests) {
     const agentPath = `${info.routePrefix}/${info.agentDownloadPath}`
     const resultPath = `${info.routePrefix}/${info.getResultPath}`
-    const host = info.functionAppUrl
+    const host = info.frontdoorUrl
 
     const agentUrl = new URL(host)
     agentUrl.pathname = agentPath
@@ -42,6 +43,24 @@ async function main() {
   }
 
   if (hasError) {
+    throw new Error('One or more tests failed')
+  }
+}
+
+async function main() {
+  const policy = retry(handleAll, {
+    backoff: new ExponentialBackoff({
+      // 5 minutes
+      maxDelay: 1000 * 60 * 5,
+    }),
+    maxAttempts: 5,
+  })
+
+  try {
+    await policy.execute(doMockTests)
+  } catch (e) {
+    console.error(e)
+
     process.exit(1)
   }
 }