dtourolle
f3fa45f742
🏗️ Build and Test JellyTau / Run Tests (pull_request) Successful in 22m12s
🏗️ Build and Test JellyTau / Supply Chain (pull_request) Successful in 37s
Traceability Validation / Check Requirement Traces (pull_request) Successful in 11s
🏗️ Build and Test JellyTau / Android Compile Check (pull_request) Successful in 4m10s
The app forgot everything it did the moment it exited. The Rust half
logged through env_logger to stdout only -- invisible to anyone who
launched from a desktop icon, and on Android worse than that: stdout is
not logcat, so the backend produced no visible output at all on the
platform carrying this project's hardest bugs. The autoplay deadlock,
the truncated-stream restart and the background-audio stall were all
diagnosed by talking a user through `adb logcat`, because there was no
other way to see anything. A panic left nothing behind at all.
Logs now go to a size-capped rotating file, to logcat on Android, and to
the webview console in dev. A panic is recorded with its backtrace before
the process dies. The frontend's messages are forwarded into the same
file, so one timeline holds both halves of the app in order -- which is
what makes a race between them legible after the fact, and races between
them are the expensive bug class here.
Redaction runs in the log FORMATTER, not at export time. A credential
sitting in a file on the device is already a disclosure; stripping it on
the way out would be too late. The exporter redacts a second time to
cover files written by builds that predate this. api_key, X-Emby-Token,
Authorization, "AccessToken" and Token="..." all reduce to [REDACTED],
while host, item ids and filenames are deliberately kept -- a log scrubbed
of those is one nobody can debug anything from. Server URLs keep scheme
and host and drop any embedded user:pass@.
Two things the tests caught that review would not have:
- redact_headers recursed on its own output. The replacement keeps the
header NAME, so the next call matched the same header forever; the
test died with a stack overflow. It is a forward scan now.
- The frontend forwarder used `void plugin.error(...)`. `void` discards
a promise's value but not its rejection, so in any webview without
IPC -- a unit test, SSR, a browser preview -- every log line became an
unhandled rejection. 20 of them showed up the first time coverage
ran. Each call now attaches a catch.
Only info and above cross the IPC boundary: debug is per-tick player
state and forwarding it would be thousands of calls a minute for output
nobody reads. A failing forwarder never propagates and never prevents the
console write.
Nothing is transmitted anywhere. The export writes a zip and reports its
path; the user attaches it themselves, which is also what keeps this from
becoming telemetry. An Android share intent is explicitly out of scope --
it is Kotlin work that belongs with the other native code.
The panic hook chains to the previous hook rather than replacing it,
because utils/lock.rs installs a silencing hook around tests that provoke
poisoned locks on purpose.
Spec in docs/specs/diagnostics-and-logging.md; UR-078 / DR-218 / UT-209.
Verified: 1079 frontend tests and the coverage gate, 759 Rust tests,
clippy -D warnings, svelte-check 0 errors, and cargo check for
aarch64-linux-android.
43 lines
1.0 KiB
Rust
43 lines
1.0 KiB
Rust
// Tauri commands exposed to frontend
|
|
// TRACES: UR-002, UR-003, UR-004, UR-005, UR-009, UR-011, UR-012, UR-017, UR-019, UR-025 |
|
|
// DR-015, DR-017, DR-021, DR-028
|
|
pub mod auth;
|
|
pub mod catalog;
|
|
pub mod connectivity;
|
|
pub mod conversions;
|
|
pub mod device;
|
|
pub mod diagnostics;
|
|
pub mod download;
|
|
pub mod favorites;
|
|
pub mod library;
|
|
pub mod offline;
|
|
pub mod playback_mode;
|
|
pub mod playback_reporting;
|
|
pub mod player;
|
|
pub mod playlist;
|
|
pub mod repository;
|
|
pub mod sessions;
|
|
pub mod storage;
|
|
pub mod sync;
|
|
pub mod sync_drain;
|
|
|
|
pub use auth::*;
|
|
pub use catalog::*;
|
|
pub use connectivity::*;
|
|
pub use conversions::*;
|
|
pub use device::*;
|
|
pub use diagnostics::*;
|
|
pub use download::*;
|
|
pub use library::*;
|
|
pub use offline::*;
|
|
pub use playback_mode::*;
|
|
#[allow(unused_imports)] // Used when playback_reporting is fully integrated
|
|
pub use playback_reporting::*;
|
|
pub use player::*;
|
|
pub use playlist::*;
|
|
pub use repository::{RepositoryManager, RepositoryManagerWrapper, *};
|
|
pub use sessions::*;
|
|
pub use storage::*;
|
|
pub use sync::*;
|
|
pub use sync_drain::*;
|