chore: add new logic for network address

Author: Davide Melfi

action.yaml

@@ -20,8 +20,7 @@ inputs:
     required: false
   dockerSharedNetwork:
     description: 'Docker network to attach lambda containers to'
-    required: false
-
+    required: false 
 runs:
   using: 'docker'
   image: 'Dockerfile'

src/containerized_test_runner/docker.py

@@ -259,7 +259,7 @@ def _capture(self,
         if self.entrypoint is not None:
             extra_docker_args += ["--entrypoint", self.entrypoint]
 
-        cmd = ["docker", "run", "-d", "-i", "--rm", "-p", "127.0.0.1:0:8080"]
+        cmd = ["docker", "run", "-d", "-i", "--rm", "-p", "0.0.0.0:0:8080"]
 
         if self.task_root != None:
             cmd += ["-v", "{}:/var/task".format(self.task_root)]
@@ -351,25 +351,26 @@ def _render_response(self, resp):
         return resp
 
     def _get_local_addr(self, container_id):
-        # If on a shared network, use the container's IP directly (no port mapping needed)
+        import re
+        # If on a shared network, use the container's IP on that network directly
         shared_network = os.environ.get("DOCKER_SHARED_NETWORK")
         if shared_network:
             docker_inspect_cmd = ["docker", "inspect", "-f", 
                 f'{{{{index .NetworkSettings.Networks "{shared_network}" "IPAddress"}}}}', container_id]
             docker_inspect_proc = subprocess.Popen(docker_inspect_cmd, stdout=subprocess.PIPE, stdin=subprocess.PIPE, stderr=subprocess.PIPE)
             ip_address = docker_inspect_proc.communicate()[0].decode().rstrip()
-            import re
             if ip_address and re.match(r'^\d+\.\d+\.\d+\.\d+$', ip_address):
                 self.logger.debug(f"shared network: using container IP {ip_address}")
                 return f"{ip_address}:8080"
 
-        # Fallback: use mapped host port
-        docker_port_cmd = ["docker", "port", container_id, "8080"]
-        docker_port_proc = subprocess.Popen(docker_port_cmd, stdout=subprocess.PIPE, stdin=subprocess.PIPE, stderr=subprocess.PIPE)
-        port_output = docker_port_proc.communicate()[0].decode().rstrip()
-        self.logger.debug(f"docker port output: '{port_output}'")
-        if port_output and port_output != "":
-            return port_output
+        # Fallback: use container's bridge network IP directly
+        docker_inspect_cmd = ["docker", "inspect", "-f", 
+            '{{range .NetworkSettings.Networks}}{{.IPAddress}}{{end}}', container_id]
+        docker_inspect_proc = subprocess.Popen(docker_inspect_cmd, stdout=subprocess.PIPE, stdin=subprocess.PIPE, stderr=subprocess.PIPE)
+        ip_address = docker_inspect_proc.communicate()[0].decode().rstrip()
+        self.logger.debug(f"bridge network IP: '{ip_address}'")
+        if ip_address and re.match(r'^\d+\.\d+\.\d+\.\d+$', ip_address):
+            return f"{ip_address}:8080"
 
         self.logger.warning("Could not determine container address, using localhost:8080")
         return "127.0.0.1:8080"