Hotfix: Use raw binary log pumping to solve console hangs (v1.1.9)

Author: CalaKuad1

backend/server/server_handler.py

@@ -483,12 +483,10 @@ def _run_server(self, command, env):
                 cwd=self.server_path, 
                 stdout=subprocess.PIPE, 
                 stderr=subprocess.PIPE, 
-                stdin=subprocess.PIPE, 
-                text=True, 
-                encoding='utf-8',
-                errors='replace',
-                bufsize=1, 
-                universal_newlines=True, 
+                stdin=subprocess.DEVNULL, 
+                text=False, 
+                bufsize=0, 
+                universal_newlines=False, 
                 creationflags=subprocess.CREATE_NO_WINDOW if sys.platform == "win32" else 0, 
                 env=env
             )
@@ -551,8 +549,15 @@ def _read_output(self, pipe, level):
         list_header_pattern = re.compile(r".*There are\s+\d+\s+of\s+a\s+max\s+of\s+\d+\s+players\s+online:\s*$", re.IGNORECASE)
 
         try:
-            logging.info(f"Handler: Log reader thread started for {level}")
-            for line in iter(pipe.readline, ''):
+            logging.info(f"Handler: Log reader thread started for {level} (Binary mode)")
+            # In binary mode, readline returns bytes
+            for line_bytes in iter(pipe.readline, b''):
+                try:
+                    line = line_bytes.decode('utf-8', errors='replace')
+                except Exception as de:
+                    logging.error(f"Handler: Decode error in {level}: {de}")
+                    continue
+
                 line_no_ansi = re.sub(r'\x1B(?:[@-Z\\-_]|\[[0-?]*[ -/]*[@-~])', '', line)
 
                 # Check for standard server start completion messages

changelog.txt

@@ -16,7 +16,7 @@
 - **Fixed backend restart loop on Windows** — Watchdog now correctly monitors the Electron process using a passed PID, preventing premature restarts during installations.
 - **Improved Java version mapping** — Version detection now correctly maps high-version servers (NeoForge/NeoMinecraft 26.x+) to **Java 25**.
 - **Fixed Startup Scripts (run.bat/run.sh)** — Implemented **Environment Injection** (JAVA_HOME & PATH) to force startup scripts to use the app's managed Java version instead of the system default.
-- **Fixed Portable Executable Stability** — Implemented **Industrial Strength** fixes including `stdout/stderr` redirection and extensive dependency bundling (uvicorn/fastapi) to prevent crashes in the installer version.
+- **Fixed Portable Executable Stability** — Implemented **Industrial Strength** fixes including `stdout/stderr` redirection and binary, unbuffered log pumping to prevent console hangs in the installer version.
 - **Robust Forge version handling** — Versions like "26.1.1" now correctly trigger the Java 25 requirement and use the managed environment.
 
 ### 🔧 IMPROVEMENTS: