Repository navigation
Conversation
- docs/architecture.md: document both derivation branches (HKDF vault keys incl. lshSeed and the seed[0..32] salt; BIP-44 owner key -> Smart Account), correct the key-usage table (authKey is sent as a bearer token), describe where keys live under derived-bundle-v1, and scope Argon2id to the legacy self-hosted password path. - python relay.py: the subgraph-routing comment still described Free -> Base Sepolia routing, retired in ops-1. - server auth.py: hashing the auth key protects against a DB leak, not replay. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Thanks, this is a big improvement, and correcting the "relay only sees a hash of authKey" claim matters. Two fixes before merge: 1. "only
2. Legacy password path: "takes the place of the seed in Step 2" reads as if Step 2's
Then it's good to merge. |
This PR rewrites the key derivation section of docs/architecture.md to cover all four HKDF keys and the BIP-44 owner key that controls the Smart Account. It also corrects the claim that the relay only sees a hash of authKey; the relay receives the key itself on every request. Separately, it fixes a Python comment that still described free-tier reads routing to Base Sepolia, and a server docstring that said hashing the auth key prevents replay attacks.