Repository navigation
docs(windows): document win01 autologon and the RDP console handoff - #49
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 23297625c7
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| | Telemetry | Off: user environment `ANONYMIZED_TELEMETRY=false` | | ||
| | Logs | `C:\Users\josh\.windows-mcp\server.log` and `server.error.log` | | ||
| | Sign-in | Sysinternals Autologon signs `josh` in at boot; the password is an LSA secret, not a registry value | | ||
| | RDP disconnect | Task `win01-rdp-to-console` (SYSTEM) runs `tscon` to return the session to the console, unlocked | |
There was a problem hiding this comment.
Document the unlocked-console security impact
When anyone besides the intended operator can access the Incus graphical console, autologon plus the RDP handoff exposes a permanently unlocked administrator desktop after every boot or RDP disconnect. The existing Safety impact section only discusses VM creation, and this change provides neither a warning nor commands to disable Autologon and win01-rdp-to-console, leaving operators without the risk and rollback guidance required for this runbook.
AGENTS.md reference: AGENTS.md:L30-L34
Useful? React with 👍 / 👎.
| Rotating the `josh` password requires running Autologon again with the new | ||
| password (`C:\ProgramData\win01\Autologon64.exe /accepteula josh WIN01 <password>`). |
There was a problem hiding this comment.
Update the canonical secret during password rotation
When josh is rotated on the live VM, rerunning Autologon updates only its stored credential; this procedure does not tell the operator to update win01.admin_password in services/windows/win01.sops.yaml, even though that file is the documented password source and is consumed when install media are staged. A later rebuild can therefore restore the old password and make the documented credential fail, so the rotation steps should update both the live account/Autologon and the canonical SOPS value.
AGENTS.md reference: AGENTS.md:L30-L34
Useful? React with 👍 / 👎.
Companion to GilmanLab/fleet#34.