Repair Windows updater, trust-store security and integration fixtures - #170
Conversation
|
Same-VM update: the quoted-path correction passes all 44 node binary tests; all 245 library tests pass. Additional integration suites passed before the separate dispatch trust-store failure tracked in #171. The complete Windows lane remains FAIL and this PR remains draft; no qualification credit. |
|
The complete Windows cargo test --locked rerun passed at the current repaired source (exit 0), including Windows ACL and native tunnel fixtures. Existing ignored tests are still excluded. The cumulative artifact/install attempt has started on the same VM; PR remains draft pending that result. No new allocation or qualification credit. |
|
Bounded source review completed at 61684d2: reviewed all 13 changed files, including non-Unix updater refusal, non-inheritable Windows process synchronization handles, ACL checks/private creation, platform-specific directory flushing and portable fixtures. Required test check is SUCCESS at this exact head; no new Rust compilation was performed for this review. No merge-blocking finding in the reviewed diff. This does not resolve #172 or provide Windows qualification: the clean packaged guest run and original Cargo failure diagnosis remain pending. New Cargo diagnostics and builder instrumentation are kept outside this PR. |
Owning defects
Closes #168. Closes #169. Closes #171. Part of #150.
Verification
cargo test --locked: PASS (exit 0) on the existing diagnostic VM. Existing ignored tests remain ignored; this is not qualification credit.Keep draft pending cumulative artifact/install result review. No new allocation, deployment, publication or stable designation. Source/artifact pins must be refreshed after reviewed source changes; diagnostic overlays never amend the immutable candidate or earn qualification credit.