Skip to content

[codex] Treat Windows Hello key timeout as cancellation#144

Open
SirAndrosBot wants to merge 1 commit into
masterfrom
codex/handle-winhello-timeout-cancel
Open

[codex] Treat Windows Hello key timeout as cancellation#144
SirAndrosBot wants to merge 1 commit into
masterfrom
codex/handle-winhello-timeout-cancel

Conversation

@SirAndrosBot

Copy link
Copy Markdown
Collaborator

Summary

  • Treat HRESULT_FROM_WIN32(WAIT_TIMEOUT) from NCryptFinalizeKey as a Windows Hello cancellation outcome.
  • Preserve existing timeout/error behavior for other NCrypt operations.

Why

When persistent key creation times out waiting for Windows Hello authorization, the plugin currently reports a generic system error and asks users to file a bug. The timeout should flow through the existing friendly cancellation path instead.

Closes #89.

Validation

  • git diff --check
  • Sub-agent implementation pass completed.
  • Sub-agent review pass found the first implementation was too broad; fixed by scoping timeout handling to NCryptFinalizeKey only and renaming the HRESULT constant.

Build was attempted with dotnet build src\KeePassWinHello.sln -c Release /p:DefineConstants=MONO, but this environment is missing .NET Framework 4.0 reference assemblies and the KeePass reference used by the project worktree.

@sonarqubecloud

Copy link
Copy Markdown

@SirAndrosBot SirAndrosBot marked this pull request as ready for review June 11, 2026 18:49
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.

[NCryptFinalizeKey — WAIT_TIMEOUT (0x80070102)]

2 participants