Skip to content

feat: StellarWalletProvider tests & secure secret key export#658

Open
fridaypetra55-afk wants to merge 1 commit into
ANYTECHS:mainfrom
fridaypetra55-afk:pr/435-496-stellar-wallet-tests-secure-export
Open

feat: StellarWalletProvider tests & secure secret key export#658
fridaypetra55-afk wants to merge 1 commit into
ANYTECHS:mainfrom
fridaypetra55-afk:pr/435-496-stellar-wallet-tests-secure-export

Conversation

@fridaypetra55-afk

Copy link
Copy Markdown
Contributor

#435 — StellarWalletProvider integration tests:

  • Wallet creation on first load, restoration from encrypted secureStorage
  • Failed decryption / corrupted data / missing address field handling
  • Export returns public key only; secret never written to storage
  • Concurrent importStellarKey edge cases; secureStorage fully mocked

#496 — Secure secret key export in Settings:

  • Replace window.confirm() with modal requiring typed 'I understand'
  • Re-authentication step via /api/auth/verify-session before key reveal
  • Secret key never rendered as selectable text — clipboard-copy only
  • Auto-hides after 15 seconds via exportHideTimerRef
  • Sentry breadcrumb emitted (no key value) on export trigger

Closes #435
Closes #496

ANYTECHS#435 — StellarWalletProvider integration tests:
- Wallet creation on first load, restoration from encrypted secureStorage
- Failed decryption / corrupted data / missing address field handling
- Export returns public key only; secret never written to storage
- Concurrent importStellarKey edge cases; secureStorage fully mocked

ANYTECHS#496 — Secure secret key export in Settings:
- Replace window.confirm() with modal requiring typed 'I understand'
- Re-authentication step via /api/auth/verify-session before key reveal
- Secret key never rendered as selectable text — clipboard-copy only
- Auto-hides after 15 seconds via exportHideTimerRef
- Sentry breadcrumb emitted (no key value) on export trigger

Closes ANYTECHS#435
Closes ANYTECHS#496
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.

Secret Key Export Must Require Re-Authentication Write unit tests for StellarWalletProvider wallet creation and restoration

1 participant