fix: nginx proxy_read_timeout now respects LLM_TIMEOUT setting (#182)

* fix: log tshark stderr on parse failure

Previously stderr was discarded, making it impossible to diagnose
why tshark rejected a file. Now stderr is captured and included in
both the log message and the thrown exception.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* fix: dark mode text contrast for error message and loading view

- ErrorMessage: override hardcoded dark-red title/text colors with
  light pink tones under [data-theme='dark']
- sgds-overrides: add explicit dark mode rules for .text-body and
  .text-muted so Bootstrap utility classes respect the dark theme;
  also wire --bs-body-color and --bs-secondary-color tokens

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* fix: address PR #179 review comments

- Replace StringBuilder with StringBuffer for stderrBuf to prevent race condition if join(5000) times out and background thread is still writing
- Cache isPrivate() results in computeTopAsns to avoid repeated string parsing per conversation; replace nested ternary with if-else for readability
- Remove duplicate [data-theme='dark'] .text-muted rule in sgds-overrides.css

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* fix: nginx proxy_read_timeout now respects LLM_TIMEOUT setting

The proxy_read_timeout was capped at 900s regardless of LLM_TIMEOUT,
causing nginx to drop long-running story generation requests early while
the backend continued and saved the result. Now the timeout is set to
max(memory-derived value, LLM_TIMEOUT + 60s) so nginx always waits at
least as long as the configured LLM timeout. LLM_TIMEOUT is forwarded
to the nginx container in docker-compose.yml.

Closes #181

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* fix: harden tshark stderr drainer charset, buffer cap, and logging

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: NotYuSheng

backend/src/main/java/com/tracepcap/analysis/service/PcapParserService.java

@@ -91,10 +91,14 @@ public PcapAnalysisResult analyzePcapFile(File pcapFile) {
       StringBuffer stderrBuf = new StringBuffer();
       Thread stderrThread = new Thread(() -> {
         try (BufferedReader err =
-            new BufferedReader(new InputStreamReader(process.getErrorStream()))) {
+            new BufferedReader(new InputStreamReader(process.getErrorStream(), java.nio.charset.StandardCharsets.UTF_8))) {
           String l;
-          while ((l = err.readLine()) != null) stderrBuf.append(l).append('\n');
-        } catch (Exception ignored) {}
+          while ((l = err.readLine()) != null) {
+            if (stderrBuf.length() < 10_000) stderrBuf.append(l).append('\n');
+          }
+        } catch (Exception e) {
+          log.warn("Failed to drain tshark stderr", e);
+        }
       });
       stderrThread.setDaemon(true);
       stderrThread.start();

docker-compose.yml

@@ -110,6 +110,8 @@ services:
     environment:
       # Memory allocation — nginx body limit and timeouts derived from this
       APP_MEMORY_MB: ${APP_MEMORY_MB:-2048}
+      # LLM timeout — nginx proxy_read_timeout is set to max(memory-derived, LLM_TIMEOUT+60s)
+      LLM_TIMEOUT: ${LLM_TIMEOUT:-300}
       # Timezone Configuration
       TZ: Asia/Singapore
     ports:

frontend/src/assets/styles/sgds-overrides.css

@@ -316,6 +316,11 @@
   color: var(--tp-text) !important;
 }
 
+[data-theme='dark'] .text-muted {
+  color: var(--tp-text-muted) !important;
+}
+
+
 /* Bootstrap utility classes */
 [data-theme='dark'] .bg-light,
 [data-theme='dark'] .bg-white {

nginx/docker-entrypoint.sh

@@ -9,10 +9,12 @@ MAX_UPLOAD_MB=$(( MEM * 25 / 100 ))
 # Nginx body limit = max upload + 50 MB multipart overhead buffer
 NGINX_MAX_BODY_SIZE="$(( MAX_UPLOAD_MB + 50 ))M"
 
-# Proxy timeout = 45% of APP_MEMORY_MB, clamped to [300, 900] seconds
+# Proxy timeout = max(45% of APP_MEMORY_MB, LLM_TIMEOUT + 60s buffer), floor 300s
 NGINX_PROXY_TIMEOUT=$(( MEM * 45 / 100 ))
 if [ "$NGINX_PROXY_TIMEOUT" -lt 300 ]; then NGINX_PROXY_TIMEOUT=300; fi
-if [ "$NGINX_PROXY_TIMEOUT" -gt 900 ]; then NGINX_PROXY_TIMEOUT=900; fi
+LLM_TIMEOUT_S=${LLM_TIMEOUT:-300}
+LLM_PROXY_TIMEOUT=$(( LLM_TIMEOUT_S + 60 ))
+if [ "$NGINX_PROXY_TIMEOUT" -lt "$LLM_PROXY_TIMEOUT" ]; then NGINX_PROXY_TIMEOUT=$LLM_PROXY_TIMEOUT; fi
 
 export NGINX_MAX_BODY_SIZE
 export NGINX_PROXY_TIMEOUT