fix: browser tests failing because containers can't find Redis

transport/images/js/v1.x/.aegir.js

@@ -52,7 +52,25 @@ export default {
       redisClient.on('error', (err) => {
         console.error('Redis client error:', err)
       })
-      await redisClient.connect()
+
+      // Sometimes Docker DNS is slow and Redis hostname isn't found right away
+      // so we just try a few times instead of giving up immediately
+      const maxRetries = 10
+      for (let attempt = 1; attempt <= maxRetries; attempt++) {
+        try {
+          await redisClient.connect()
+          if (attempt > 1) {
+            console.error(`Redis connected on attempt ${attempt}`)
+          }
+          break
+        } catch (err) {
+          if (attempt === maxRetries) {
+            throw new Error(`Failed to connect to Redis after ${maxRetries} attempts: ${err.message}`)
+          }
+          console.error(`Redis connection attempt ${attempt}/${maxRetries} failed: ${err.message}`)
+          await new Promise(resolve => setTimeout(resolve, attempt * 1000))
+        }
+      }
 
       const requestListener = async function (req, res) {
         let requestJSON

transport/images/js/v1.x/BrowserDockerfile

@@ -12,4 +12,12 @@ ENV BROWSER=${BROWSER}
 # container and not during the test run which slows everything down
 RUN npx playwright-test --runner mocha --browser $BROWSER --grep do-not-match-anything
 
+# Playwright install can break /etc/resolv.conf by turning it into
+# a dead symlink. If that happens, DNS lookups fail with EAI_AGAIN.
+# This just checks for that and fixes it.
+RUN if [ -L /etc/resolv.conf ]; then \
+      rm -f /etc/resolv.conf && \
+      echo "nameserver 8.8.8.8" > /etc/resolv.conf; \
+    fi
+
 ENTRYPOINT ["sh", "-c", "npm test -- -t browser -- --browser \"$BROWSER\""]

transport/images/js/v2.x/.aegir.js

@@ -52,7 +52,25 @@ export default {
       redisClient.on('error', (err) => {
         console.error('Redis client error:', err)
       })
-      await redisClient.connect()
+
+      // Sometimes Docker DNS is slow and Redis hostname isn't found right away
+      // so we just try a few times instead of giving up immediately
+      const maxRetries = 10
+      for (let attempt = 1; attempt <= maxRetries; attempt++) {
+        try {
+          await redisClient.connect()
+          if (attempt > 1) {
+            console.error(`Redis connected on attempt ${attempt}`)
+          }
+          break
+        } catch (err) {
+          if (attempt === maxRetries) {
+            throw new Error(`Failed to connect to Redis after ${maxRetries} attempts: ${err.message}`)
+          }
+          console.error(`Redis connection attempt ${attempt}/${maxRetries} failed: ${err.message}`)
+          await new Promise(resolve => setTimeout(resolve, attempt * 1000))
+        }
+      }
 
       const requestListener = async function (req, res) {
         let requestJSON

transport/images/js/v2.x/BrowserDockerfile

@@ -12,4 +12,12 @@ ENV BROWSER=${BROWSER}
 # container and not during the test run which slows everything down
 RUN npx playwright-test --runner mocha --browser $BROWSER --grep do-not-match-anything
 
+# Playwright install can break /etc/resolv.conf by turning it into
+# a dead symlink. If that happens, DNS lookups fail with EAI_AGAIN.
+# This just checks for that and fixes it.
+RUN if [ -L /etc/resolv.conf ]; then \
+      rm -f /etc/resolv.conf && \
+      echo "nameserver 8.8.8.8" > /etc/resolv.conf; \
+    fi
+
 ENTRYPOINT ["sh", "-c", "npm test -- -t browser -- --browser \"$BROWSER\""]

transport/images/js/v3.x/.aegir.js

@@ -49,9 +49,26 @@ export default {
         console.error('Redis client error:', err)
       })
 
+      // Sometimes Docker DNS is slow and Redis hostname isn't found right away
+      // so we just try a few times instead of giving up immediately
+      const maxRetries = 10
       let start = Date.now()
       console.error('connect redis client')
-      await redisClient.connect()
+      for (let attempt = 1; attempt <= maxRetries; attempt++) {
+        try {
+          await redisClient.connect()
+          if (attempt > 1) {
+            console.error(`Redis connected on attempt ${attempt}`)
+          }
+          break
+        } catch (err) {
+          if (attempt === maxRetries) {
+            throw new Error(`Failed to connect to Redis after ${maxRetries} attempts: ${err.message}`)
+          }
+          console.error(`Redis connection attempt ${attempt}/${maxRetries} failed: ${err.message}`)
+          await new Promise(resolve => setTimeout(resolve, attempt * 1000))
+        }
+      }
       console.error('connected redis client after', Date.now() - start, 'ms')
 
       const requestListener = async function (req, res) {

transport/images/js/v3.x/BrowserDockerfile

@@ -12,4 +12,12 @@ ENV BROWSER=${BROWSER}
 # container and not during the test run which slows everything down
 RUN npx playwright-test --runner mocha --browser $BROWSER --grep do-not-match-anything
 
+# Playwright install can break /etc/resolv.conf by turning it into
+# a dead symlink. If that happens, DNS lookups fail with EAI_AGAIN.
+# This just checks for that and fixes it.
+RUN if [ -L /etc/resolv.conf ]; then \
+      rm -f /etc/resolv.conf && \
+      echo "nameserver 8.8.8.8" > /etc/resolv.conf; \
+    fi
+
 ENTRYPOINT ["sh", "-c", "npm test -- -t browser -- --browser \"$BROWSER\""]

transport/lib/run-single-test.sh

@@ -104,6 +104,8 @@ services:
     image: "${LISTENER_IMAGE}"
     container_name: ${TEST_SLUG}_listener
     init: true
+    dns:
+      - 127.0.0.11
     networks:
       - transport-network
     environment:
@@ -112,6 +114,9 @@ ${LISTENER_ENV}
   dialer:
     image: "${DIALER_IMAGE}"
     container_name: ${TEST_SLUG}_dialer
+    init: true
+    dns:
+      - 127.0.0.11
     depends_on:
       - listener
     networks: