From d247bda54d32048a205f6cacc938856f0ba6347a Mon Sep 17 00:00:00 2001 From: riccardom Date: Mon, 14 Sep 2026 09:19:25 +0200 Subject: [PATCH] [client] Stop documenting a missing published destination as an upload failure WAILS-API.md listed "a deployment that publishes no upload service" among the causes of uploadFailureReason. That was true of an earlier revision; the resolver now falls back to the service NetBird runs, so the case never produces a failure. Reserve the field for a destination that rejects the upload or cannot be reached. Reported by CodeRabbit on #7514. --- client/ui/frontend/WAILS-API.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/client/ui/frontend/WAILS-API.md b/client/ui/frontend/WAILS-API.md index 285121fab..44a20d0c8 100644 --- a/client/ui/frontend/WAILS-API.md +++ b/client/ui/frontend/WAILS-API.md @@ -148,7 +148,7 @@ Debug.RevealFile(path: string): Promise // OS file-manager focus **Log level case sensitivity bug:** `proto.LogLevel_value` is keyed on uppercase enum names (`"TRACE"`, `"DEBUG"`, `"INFO"`, `"WARN"`, `"ERROR"`, `"PANIC"`, `"FATAL"`, `"UNKNOWN"`). `Debug.SetLogLevel` calls `proto.LogLevel_value[lvl.Level]` and falls back to `INFO` on miss. `useDebugBundle` currently passes `"trace"` (lowercase), which silently maps to `INFO` — the trace-capture flow doesn't actually raise the log level today. To raise to trace, pass `{ level: "TRACE" }`. Fix on the cleanup list. -`Debug.Bundle` uploads when `upload` is true; the daemon picks the destination from what the management server publishes, so the UI never names one. Result fields: `path` (local copy), `uploadedKey` (set on success), `uploadFailureReason` (set on upload failure, including a deployment that publishes no upload service — the local copy is still saved). +`Debug.Bundle` uploads when `upload` is true; the daemon picks the destination from what the management server publishes, falling back to the service NetBird runs when it publishes none, so the UI never names one. Result fields: `path` (local copy), `uploadedKey` (set on success), `uploadFailureReason` (set when the resolved destination rejects the upload or cannot be reached — the local copy is still saved). ## `Update`