Skip to content

feat(doctor): expand wildcard exposure checks - #60

Merged
Mapika merged 1 commit into
Mapika:mainfrom
Guflly:feat/doctor-sensitive-processes
Jul 31, 2026
Merged

feat(doctor): expand wildcard exposure checks#60
Mapika merged 1 commit into
Mapika:mainfrom
Guflly:feat/doctor-sensitive-processes

Conversation

@Guflly

@Guflly Guflly commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Adds the database and messaging-service executables listed in #51 to the wildcard exposure check, with coverage for case-insensitive matching.

Tests:

  • cargo fmt --check
  • cargo test doctor::tests
  • Windows, Linux, and macOS target checks

The full Windows suite still hits the existing far-future FILETIME panic on main.

Closes #51

@Mapika
Mapika merged commit 443d2d0 into Mapika:main Jul 31, 2026
6 checks passed
Mapika added a commit that referenced this pull request Jul 31, 2026
UNIX_EPOCH + Duration panics when the result is unrepresentable. A Windows
SystemTime is FILETIME-backed and ends near year 30828, so a garbage process
creation time overflows it and aborts the scan.

The suite never caught this: `cargo test` only runs in the Linux `quality`
job, so the `#[cfg(windows)]` and `#[cfg(macos)]` test modules compile but
never execute. Run the suite on the natively-runnable platform targets too.

Reported by @Guflly in #60.
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.

doctor: broaden the wildcard-exposure check beyond six databases

2 participants