Skip to content

fix(service): report running after secure control pipe#124

Merged
bestlux merged 2 commits into
mainfrom
codex/fix-msi-service-start-diagnostics
Jun 23, 2026
Merged

fix(service): report running after secure control pipe#124
bestlux merged 2 commits into
mainfrom
codex/fix-msi-service-start-diagnostics

Conversation

@bestlux

@bestlux bestlux commented Jun 23, 2026

Copy link
Copy Markdown
Owner

Intent

Fix the MSI-owned Windows service-start path so SCM can observe BoundlessService as Running after the secure control named pipe is available, without weakening the allowed-user SID boundary.

Changes

  • Split daemon startup into runtime preparation and background task startup.
  • In Windows service mode, bind the allowed-user secure named pipe and report SCM Running before starting slower/background runtime tasks.
  • Keep service-session input in the unsupported diagnostic mode with no active capture target.
  • Add ProgramData startup diagnostics at C:\ProgramData\Boundless\logs\boundless-service-startup.log so failures survive MSI rollback.
  • Extend the focused service-mode regression to exercise prepare -> secure named-pipe bind -> background task startup ordering.

Validation

  • cargo fmt --all -- --check
  • CARGO_INCREMENTAL=0 cargo test -p boundless-daemon service_mode_named_pipe_startup_preserves_diagnostic_input_state -- --nocapture
  • CARGO_INCREMENTAL=0 cargo test -p boundless-daemon service_session -- --nocapture
  • CARGO_INCREMENTAL=0 cargo clippy --workspace --all-targets -- -D warnings
  • CARGO_INCREMENTAL=0 cargo test --workspace --quiet
  • CARGO_INCREMENTAL=0 cargo build --release -p boundless-daemon --bin boundless-service
  • git diff --check

Elevated validation

This worker shell is not elevated (CODY-PC\iomancer, Administrator=false), so I did not run SCM/MSI install smoke here and did not force UAC. A real elevated service-start smoke is still required before another dogfood package is built.

@bestlux bestlux merged commit e89e5d0 into main Jun 23, 2026
6 checks passed
@bestlux bestlux deleted the codex/fix-msi-service-start-diagnostics branch June 23, 2026 15:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant