Skip to content

Update info about Android TEE and StrongBox#961

Merged
BrandtH22 merged 2 commits into
mainfrom
2026_04_07_signer
Apr 24, 2026
Merged

Update info about Android TEE and StrongBox#961
BrandtH22 merged 2 commits into
mainfrom
2026_04_07_signer

Conversation

@danieljperry
Copy link
Copy Markdown
Contributor

@danieljperry danieljperry commented Apr 7, 2026

Note

Low Risk
Low risk: documentation-only updates clarifying Android key storage options and a recovery edge case; no runtime or security-sensitive code changes.

Overview
Updates Signer documentation to reflect the current UI flow for adding keys (via the ... menu) and expands Android guidance to distinguish StrongBox vs TEE vs software fallback, including an explicit note about current in-app messaging limitations and a new FAQ on whether TEE is sufficient.

Extends the Cloud Wallet FAQ with a new recovery scenario clarifying that once a recovery is initiated, the replacement keys are locked in and can’t be swapped without cancelling and re-initiating recovery.

Reviewed by Cursor Bugbot for commit ec086bd. Bugbot is set up for automated code reviews on this repo. Configure here.

hoffmang9
hoffmang9 previously approved these changes Apr 7, 2026
Copy link
Copy Markdown
Member

@hoffmang9 hoffmang9 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgmt

Comment thread docs/chia-signer/faq.md
Comment thread docs/chia-signer/faq.md
Comment thread docs/chia-signer/faq.md
Copy link
Copy Markdown
Contributor

@BrandtH22 BrandtH22 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@BrandtH22 BrandtH22 merged commit 5ac2ff7 into main Apr 24, 2026
15 checks passed
@BrandtH22 BrandtH22 deleted the 2026_04_07_signer branch April 24, 2026 14:26
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.

3 participants