Update app.py

app.py

@@ -171,37 +171,67 @@ def start_ngrok_tunnel(port=25565):
     
     try:
         # Start ngrok in background
+        print("Launching ngrok process...")
         process = subprocess.Popen([
             ngrok_path, "tcp", str(port), "--log", "stdout"
         ], stdout=subprocess.PIPE, stderr=subprocess.PIPE, text=True)
         
-        # Give ngrok a moment to start
+        # Give ngrok more time to start and wait for API to be available
         import time
-        time.sleep(3)
+        import json
+        import urllib.request
         
-        # Try to get the tunnel URL from ngrok API
-        try:
-            import json
-            import urllib.request
+        print("Waiting for ngrok to initialize...")
+        max_retries = 30  # Try for up to 30 seconds
+        retry_count = 0
+        
+        while retry_count < max_retries:
+            time.sleep(1)
+            retry_count += 1
             
-            # ngrok exposes a local API on port 4040
-            api_url = "http://localhost:4040/api/tunnels"
-            with urllib.request.urlopen(api_url) as response:
-                data = json.loads(response.read().decode())
+            try:
+                # Check if ngrok process is still running
+                if process.poll() is not None:
+                    # Process has ended, check for errors
+                    stdout, stderr = process.communicate()
+                    print(f"ngrok process ended unexpectedly:")
+                    print(f"STDOUT: {stdout}")
+                    print(f"STDERR: {stderr}")
+                    return None, None
                 
-            if data.get('tunnels'):
-                tunnel_url = data['tunnels'][0]['public_url']
-                print(f"✓ ngrok tunnel active: {tunnel_url}")
-                print(f"Players can connect to: {tunnel_url.replace('tcp://', '')}")
-                return process, tunnel_url
-            else:
-                print("No active tunnels found")
-                return process, None
-                
-        except Exception as e:
-            print(f"Could not retrieve tunnel URL: {e}")
-            print("Check ngrok dashboard at http://localhost:4040")
+                # Try to connect to ngrok API
+                api_url = "http://localhost:4040/api/tunnels"
+                with urllib.request.urlopen(api_url, timeout=2) as response:
+                    data = json.loads(response.read().decode())
+                    
+                if data.get('tunnels'):
+                    tunnel_url = data['tunnels'][0]['public_url']
+                    print(f"✓ ngrok tunnel active: {tunnel_url}")
+                    print(f"Players can connect to: {tunnel_url.replace('tcp://', '')}")
+                    return process, tunnel_url
+                else:
+                    print(f"Waiting for tunnel... (attempt {retry_count}/{max_retries})")
+                    
+            except urllib.error.URLError as e:
+                if retry_count % 5 == 0:  # Print every 5 seconds
+                    print(f"Waiting for ngrok API... (attempt {retry_count}/{max_retries})")
+                continue
+            except Exception as e:
+                print(f"Error checking ngrok status: {e}")
+                continue
+        
+        # If we get here, we timed out
+        print("Timeout waiting for ngrok to start. Checking process status...")
+        if process.poll() is None:
+            print("ngrok process is still running but API is not responding.")
+            print("You may need to check the ngrok dashboard manually at http://localhost:4040")
             return process, None
+        else:
+            stdout, stderr = process.communicate()
+            print(f"ngrok process ended during startup:")
+            print(f"STDOUT: {stdout}")
+            print(f"STDERR: {stderr}")
+            return None, None
             
     except Exception as e:
         print(f"Failed to start ngrok tunnel: {e}")