@aztec/key-store 0.0.1-commit.e558bd1c → 0.0.1-commit.e57c76e
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/dest/key_store.d.ts +14 -5
- package/dest/key_store.d.ts.map +1 -1
- package/dest/key_store.js +93 -60
- package/package.json +5 -5
- package/src/key_store.ts +111 -64
package/dest/key_store.d.ts
CHANGED
|
@@ -30,6 +30,8 @@ export declare class KeyStore {
|
|
|
30
30
|
* @returns A Promise that resolves to an array of account addresses.
|
|
31
31
|
*/
|
|
32
32
|
getAccounts(): Promise<AztecAddress[]>;
|
|
33
|
+
/** Checks whether an account is registered in the key store. */
|
|
34
|
+
hasAccount(account: AztecAddress): Promise<boolean>;
|
|
33
35
|
/**
|
|
34
36
|
* Gets the key validation request for a given master public key hash and contract address.
|
|
35
37
|
* @throws If the account corresponding to the master public key hash does not exist in the key store.
|
|
@@ -82,13 +84,20 @@ export declare class KeyStore {
|
|
|
82
84
|
*/
|
|
83
85
|
getAppOutgoingViewingSecretKey(account: AztecAddress, app: AztecAddress): Promise<Fr>;
|
|
84
86
|
/**
|
|
85
|
-
* Retrieves the sk_m corresponding to the pk_m.
|
|
86
|
-
* @throws If the provided
|
|
87
|
-
* @param
|
|
87
|
+
* Retrieves the sk_m corresponding to the given pk_m hash.
|
|
88
|
+
* @throws If the provided hash is not associated with any of the registered accounts.
|
|
89
|
+
* @param pkMHash - The master public key hash to get secret key for.
|
|
88
90
|
* @returns A Promise that resolves to sk_m.
|
|
89
91
|
* @dev Used when feeding the sk_m to the kernel circuit for keys verification.
|
|
90
92
|
*/
|
|
91
|
-
getMasterSecretKey(
|
|
93
|
+
getMasterSecretKey(pkMHash: Fr): Promise<GrumpkinScalar>;
|
|
94
|
+
/**
|
|
95
|
+
* Checks whether a given account has a key matching the provided master public key hash.
|
|
96
|
+
* @param account - The account address to check.
|
|
97
|
+
* @param pkMHash - The master public key hash to look for.
|
|
98
|
+
* @returns True if the account has a key with the given hash.
|
|
99
|
+
*/
|
|
100
|
+
accountHasKey(account: AztecAddress, pkMHash: Fr): Promise<boolean>;
|
|
92
101
|
/**
|
|
93
102
|
* Gets the key prefix and account address for a given value.
|
|
94
103
|
* @returns A tuple containing the key prefix and account address.
|
|
@@ -97,4 +106,4 @@ export declare class KeyStore {
|
|
|
97
106
|
*/
|
|
98
107
|
getKeyPrefixAndAccount(value: Bufferable): Promise<[KeyPrefix, AztecAddress]>;
|
|
99
108
|
}
|
|
100
|
-
//# sourceMappingURL=data:application/json;base64,
|
|
109
|
+
//# sourceMappingURL=data:application/json;base64,eyJ2ZXJzaW9uIjozLCJmaWxlIjoia2V5X3N0b3JlLmQudHMiLCJzb3VyY2VSb290IjoiIiwic291cmNlcyI6WyIuLi9zcmMva2V5X3N0b3JlLnRzIl0sIm5hbWVzIjpbXSwibWFwcGluZ3MiOiJBQUVBLE9BQU8sRUFBRSxFQUFFLEVBQUUsTUFBTSxnQ0FBZ0MsQ0FBQztBQUNwRCxPQUFPLEVBQUUsY0FBYyxFQUFTLE1BQU0sbUNBQW1DLENBQUM7QUFFMUUsT0FBTyxFQUFFLEtBQUssVUFBVSxFQUFxQixNQUFNLDZCQUE2QixDQUFDO0FBQ2pGLE9BQU8sS0FBSyxFQUFFLGlCQUFpQixFQUFpQixNQUFNLGlCQUFpQixDQUFDO0FBQ3hFLE9BQU8sRUFBRSxZQUFZLEVBQUUsTUFBTSw2QkFBNkIsQ0FBQztBQUMzRCxPQUFPLEVBQUUsZUFBZSxFQUFFLEtBQUssY0FBYyxFQUFFLE1BQU0sd0JBQXdCLENBQUM7QUFDOUUsT0FBTyxFQUFFLG9CQUFvQixFQUFFLE1BQU0sc0JBQXNCLENBQUM7QUFDNUQsT0FBTyxFQUVMLEtBQUssU0FBUyxFQUNkLEtBQUssU0FBUyxFQUtmLE1BQU0sb0JBQW9CLENBQUM7QUFPNUI7O0dBRUc7QUFDSCxxQkFBYSxRQUFROztJQUNuQixnQkFBdUIsY0FBYyxLQUFLO0lBSTFDLFlBQVksUUFBUSxFQUFFLGlCQUFpQixFQUd0QztJQUVEOzs7T0FHRztJQUNJLGFBQWEsSUFBSSxPQUFPLENBQUMsZUFBZSxDQUFDLENBSS9DO0lBRUQ7Ozs7O09BS0c7SUFDVSxVQUFVLENBQUMsRUFBRSxFQUFFLEVBQUUsRUFBRSxjQUFjLEVBQUUsY0FBYyxHQUFHLE9BQU8sQ0FBQyxlQUFlLENBQUMsQ0ErQ3hGO0lBRUQ7OztPQUdHO0lBQ1UsV0FBVyxJQUFJLE9BQU8sQ0FBQyxZQUFZLEVBQUUsQ0FBQyxDQUtsRDtJQUVELGdFQUFnRTtJQUNuRCxVQUFVLENBQUMsT0FBTyxFQUFFLFlBQVksR0FBRyxPQUFPLENBQUMsT0FBTyxDQUFDLENBRS9EO0lBRUQ7Ozs7OztPQU1HO0lBQ0ksdUJBQXVCLENBQUMsT0FBTyxFQUFFLEVBQUUsRUFBRSxlQUFlLEVBQUUsWUFBWSxHQUFHLE9BQU8sQ0FBQyxvQkFBb0IsQ0FBQyxDQTBDeEc7SUFFRDs7Ozs7T0FLRztJQUNVLDJCQUEyQixDQUFDLE9BQU8sRUFBRSxZQUFZLEdBQUcsT0FBTyxDQUFDLFNBQVMsQ0FBQyxDQVFsRjtJQUVEOzs7OztPQUtHO0lBQ1UsaUNBQWlDLENBQUMsT0FBTyxFQUFFLFlBQVksR0FBRyxPQUFPLENBQUMsU0FBUyxDQUFDLENBUXhGO0lBRUQ7Ozs7O09BS0c7SUFDVSxpQ0FBaUMsQ0FBQyxPQUFPLEVBQUUsWUFBWSxHQUFHLE9BQU8sQ0FBQyxTQUFTLENBQUMsQ0FReEY7SUFFRDs7Ozs7T0FLRztJQUNVLHlCQUF5QixDQUFDLE9BQU8sRUFBRSxZQUFZLEdBQUcsT0FBTyxDQUFDLFNBQVMsQ0FBQyxDQVFoRjtJQUVEOzs7OztPQUtHO0lBQ1UsaUNBQWlDLENBQUMsT0FBTyxFQUFFLFlBQVksR0FBRyxPQUFPLENBQUMsY0FBYyxDQUFDLENBUTdGO0lBRUQ7Ozs7OztPQU1HO0lBQ1UsOEJBQThCLENBQUMsT0FBTyxFQUFFLFlBQVksRUFBRSxHQUFHLEVBQUUsWUFBWSxHQUFHLE9BQU8sQ0FBQyxFQUFFLENBQUMsQ0Fhakc7SUFFRDs7Ozs7O09BTUc7SUFDSSxrQkFBa0IsQ0FBQyxPQUFPLEVBQUUsRUFBRSxHQUFHLE9BQU8sQ0FBQyxjQUFjLENBQUMsQ0E2QjlEO0lBRUQ7Ozs7O09BS0c7SUFDSSxhQUFhLENBQUMsT0FBTyxFQUFFLFlBQVksRUFBRSxPQUFPLEVBQUUsRUFBRSxHQUFHLE9BQU8sQ0FBQyxPQUFPLENBQUMsQ0FXekU7SUFFRDs7Ozs7T0FLRztJQUNVLHNCQUFzQixDQUFDLEtBQUssRUFBRSxVQUFVLEdBQUcsT0FBTyxDQUFDLENBQUMsU0FBUyxFQUFFLFlBQVksQ0FBQyxDQUFDLENBY3pGO0NBQ0YifQ==
|
package/dest/key_store.d.ts.map
CHANGED
|
@@ -1 +1 @@
|
|
|
1
|
-
{"version":3,"file":"key_store.d.ts","sourceRoot":"","sources":["../src/key_store.ts"],"names":[],"mappings":"AAEA,OAAO,EAAE,EAAE,EAAE,MAAM,gCAAgC,CAAC;AACpD,OAAO,EAAE,cAAc,EAAS,MAAM,mCAAmC,CAAC;AAE1E,OAAO,EAAE,KAAK,UAAU,EAAqB,MAAM,6BAA6B,CAAC;AACjF,OAAO,KAAK,EAAE,iBAAiB,EAAiB,MAAM,iBAAiB,CAAC;AACxE,OAAO,EAAE,YAAY,EAAE,MAAM,6BAA6B,CAAC;AAC3D,OAAO,EAAE,eAAe,EAAE,KAAK,cAAc,EAAE,MAAM,wBAAwB,CAAC;AAC9E,OAAO,EAAE,oBAAoB,EAAE,MAAM,sBAAsB,CAAC;AAC5D,OAAO,EAEL,KAAK,SAAS,EACd,KAAK,SAAS,
|
|
1
|
+
{"version":3,"file":"key_store.d.ts","sourceRoot":"","sources":["../src/key_store.ts"],"names":[],"mappings":"AAEA,OAAO,EAAE,EAAE,EAAE,MAAM,gCAAgC,CAAC;AACpD,OAAO,EAAE,cAAc,EAAS,MAAM,mCAAmC,CAAC;AAE1E,OAAO,EAAE,KAAK,UAAU,EAAqB,MAAM,6BAA6B,CAAC;AACjF,OAAO,KAAK,EAAE,iBAAiB,EAAiB,MAAM,iBAAiB,CAAC;AACxE,OAAO,EAAE,YAAY,EAAE,MAAM,6BAA6B,CAAC;AAC3D,OAAO,EAAE,eAAe,EAAE,KAAK,cAAc,EAAE,MAAM,wBAAwB,CAAC;AAC9E,OAAO,EAAE,oBAAoB,EAAE,MAAM,sBAAsB,CAAC;AAC5D,OAAO,EAEL,KAAK,SAAS,EACd,KAAK,SAAS,EAKf,MAAM,oBAAoB,CAAC;AAO5B;;GAEG;AACH,qBAAa,QAAQ;;IACnB,gBAAuB,cAAc,KAAK;IAI1C,YAAY,QAAQ,EAAE,iBAAiB,EAGtC;IAED;;;OAGG;IACI,aAAa,IAAI,OAAO,CAAC,eAAe,CAAC,CAI/C;IAED;;;;;OAKG;IACU,UAAU,CAAC,EAAE,EAAE,EAAE,EAAE,cAAc,EAAE,cAAc,GAAG,OAAO,CAAC,eAAe,CAAC,CA+CxF;IAED;;;OAGG;IACU,WAAW,IAAI,OAAO,CAAC,YAAY,EAAE,CAAC,CAKlD;IAED,gEAAgE;IACnD,UAAU,CAAC,OAAO,EAAE,YAAY,GAAG,OAAO,CAAC,OAAO,CAAC,CAE/D;IAED;;;;;;OAMG;IACI,uBAAuB,CAAC,OAAO,EAAE,EAAE,EAAE,eAAe,EAAE,YAAY,GAAG,OAAO,CAAC,oBAAoB,CAAC,CA0CxG;IAED;;;;;OAKG;IACU,2BAA2B,CAAC,OAAO,EAAE,YAAY,GAAG,OAAO,CAAC,SAAS,CAAC,CAQlF;IAED;;;;;OAKG;IACU,iCAAiC,CAAC,OAAO,EAAE,YAAY,GAAG,OAAO,CAAC,SAAS,CAAC,CAQxF;IAED;;;;;OAKG;IACU,iCAAiC,CAAC,OAAO,EAAE,YAAY,GAAG,OAAO,CAAC,SAAS,CAAC,CAQxF;IAED;;;;;OAKG;IACU,yBAAyB,CAAC,OAAO,EAAE,YAAY,GAAG,OAAO,CAAC,SAAS,CAAC,CAQhF;IAED;;;;;OAKG;IACU,iCAAiC,CAAC,OAAO,EAAE,YAAY,GAAG,OAAO,CAAC,cAAc,CAAC,CAQ7F;IAED;;;;;;OAMG;IACU,8BAA8B,CAAC,OAAO,EAAE,YAAY,EAAE,GAAG,EAAE,YAAY,GAAG,OAAO,CAAC,EAAE,CAAC,CAajG;IAED;;;;;;OAMG;IACI,kBAAkB,CAAC,OAAO,EAAE,EAAE,GAAG,OAAO,CAAC,cAAc,CAAC,CA6B9D;IAED;;;;;OAKG;IACI,aAAa,CAAC,OAAO,EAAE,YAAY,EAAE,OAAO,EAAE,EAAE,GAAG,OAAO,CAAC,OAAO,CAAC,CAWzE;IAED;;;;;OAKG;IACU,sBAAsB,CAAC,KAAK,EAAE,UAAU,GAAG,OAAO,CAAC,CAAC,SAAS,EAAE,YAAY,CAAC,CAAC,CAczF;CACF"}
|
package/dest/key_store.js
CHANGED
|
@@ -1,4 +1,4 @@
|
|
|
1
|
-
import {
|
|
1
|
+
import { DomainSeparator } from '@aztec/constants';
|
|
2
2
|
import { poseidon2HashWithSeparator } from '@aztec/foundation/crypto/poseidon';
|
|
3
3
|
import { Fr } from '@aztec/foundation/curves/bn254';
|
|
4
4
|
import { GrumpkinScalar, Point } from '@aztec/foundation/curves/grumpkin';
|
|
@@ -7,7 +7,7 @@ import { serializeToBuffer } from '@aztec/foundation/serialize';
|
|
|
7
7
|
import { AztecAddress } from '@aztec/stdlib/aztec-address';
|
|
8
8
|
import { CompleteAddress } from '@aztec/stdlib/contract';
|
|
9
9
|
import { KeyValidationRequest } from '@aztec/stdlib/kernel';
|
|
10
|
-
import { KEY_PREFIXES, computeAppSecretKey, deriveKeys, derivePublicKeyFromSecretKey } from '@aztec/stdlib/keys';
|
|
10
|
+
import { KEY_PREFIXES, computeAppSecretKey, deriveKeys, derivePublicKeyFromSecretKey, hashPublicKey } from '@aztec/stdlib/keys';
|
|
11
11
|
/** Maps a key prefix to the storage suffix for the corresponding master secret key. */ function secretKeyStorageSuffix(prefix) {
|
|
12
12
|
return prefix === 'n' ? 'nhk_m' : `${prefix}sk_m`;
|
|
13
13
|
}
|
|
@@ -35,30 +35,32 @@ import { KEY_PREFIXES, computeAppSecretKey, deriveKeys, derivePublicKeyFromSecre
|
|
|
35
35
|
* @param partialAddress - The partial address of the account.
|
|
36
36
|
* @returns The account's complete address.
|
|
37
37
|
*/ async addAccount(sk, partialAddress) {
|
|
38
|
-
const { masterNullifierHidingKey, masterIncomingViewingSecretKey, masterOutgoingViewingSecretKey, masterTaggingSecretKey, publicKeys } = await deriveKeys(sk);
|
|
39
|
-
const completeAddress = await CompleteAddress.
|
|
38
|
+
const { masterNullifierHidingKey, masterIncomingViewingSecretKey, masterOutgoingViewingSecretKey, masterTaggingSecretKey, masterNullifierPublicKey, masterOutgoingViewingPublicKey, masterTaggingPublicKey, publicKeys } = await deriveKeys(sk);
|
|
39
|
+
const completeAddress = await CompleteAddress.fromPublicKeysAndPartialAddress(publicKeys, partialAddress);
|
|
40
40
|
const { address: account } = completeAddress;
|
|
41
|
-
//
|
|
42
|
-
|
|
43
|
-
|
|
44
|
-
|
|
45
|
-
const
|
|
41
|
+
// The kernel cannot check that nhpk/ovpk/tpk are on-curve or non-infinity, so the PXE/key-store
|
|
42
|
+
// must guarantee it before persistence. By design, the above derivation produces points that are on
|
|
43
|
+
// the curve and not at infinity.
|
|
44
|
+
// The npk/ovpk/tpk hashes are already in publicKeys; ivpk_m_hash is computed for indexing.
|
|
45
|
+
const masterIncomingViewingPublicKeyHash = await hashPublicKey(publicKeys.ivpkM);
|
|
46
|
+
// The Message Signing and Fallback Keys don't have a derivation path yet, so we just use the default values for their hashes.
|
|
47
|
+
// So we avoid storing them persistently. The default hash is still required for address derivation
|
|
46
48
|
await this.#db.transactionAsync(async ()=>{
|
|
47
49
|
// Naming of keys is as follows ${account}-${n/iv/ov/t}${sk/pk}_m
|
|
48
50
|
await this.#keys.set(`${account.toString()}-ivsk_m`, masterIncomingViewingSecretKey.toBuffer());
|
|
49
51
|
await this.#keys.set(`${account.toString()}-ovsk_m`, masterOutgoingViewingSecretKey.toBuffer());
|
|
50
52
|
await this.#keys.set(`${account.toString()}-tsk_m`, masterTaggingSecretKey.toBuffer());
|
|
51
53
|
await this.#keys.set(`${account.toString()}-nhk_m`, masterNullifierHidingKey.toBuffer());
|
|
52
|
-
await this.#keys.set(`${account.toString()}-npk_m`,
|
|
53
|
-
await this.#keys.set(`${account.toString()}-ivpk_m`, publicKeys.
|
|
54
|
-
await this.#keys.set(`${account.toString()}-ovpk_m`,
|
|
55
|
-
await this.#keys.set(`${account.toString()}-tpk_m`,
|
|
54
|
+
await this.#keys.set(`${account.toString()}-npk_m`, masterNullifierPublicKey.toBuffer());
|
|
55
|
+
await this.#keys.set(`${account.toString()}-ivpk_m`, publicKeys.ivpkM.toBuffer());
|
|
56
|
+
await this.#keys.set(`${account.toString()}-ovpk_m`, masterOutgoingViewingPublicKey.toBuffer());
|
|
57
|
+
await this.#keys.set(`${account.toString()}-tpk_m`, masterTaggingPublicKey.toBuffer());
|
|
56
58
|
// We store pk_m_hash under `account-{n/iv/ov/t}pk_m_hash` key to be able to obtain address and key prefix
|
|
57
59
|
// using the #getKeyPrefixAndAccount function later on
|
|
58
|
-
await this.#keys.set(`${account.toString()}-npk_m_hash`,
|
|
60
|
+
await this.#keys.set(`${account.toString()}-npk_m_hash`, publicKeys.npkMHash.toBuffer());
|
|
59
61
|
await this.#keys.set(`${account.toString()}-ivpk_m_hash`, masterIncomingViewingPublicKeyHash.toBuffer());
|
|
60
|
-
await this.#keys.set(`${account.toString()}-ovpk_m_hash`,
|
|
61
|
-
await this.#keys.set(`${account.toString()}-tpk_m_hash`,
|
|
62
|
+
await this.#keys.set(`${account.toString()}-ovpk_m_hash`, publicKeys.ovpkMHash.toBuffer());
|
|
63
|
+
await this.#keys.set(`${account.toString()}-tpk_m_hash`, publicKeys.tpkMHash.toBuffer());
|
|
62
64
|
});
|
|
63
65
|
// At last, we return the newly derived account address
|
|
64
66
|
return completeAddress;
|
|
@@ -72,39 +74,45 @@ import { KEY_PREFIXES, computeAppSecretKey, deriveKeys, derivePublicKeyFromSecre
|
|
|
72
74
|
const accounts = allMapKeys.filter((key)=>key.endsWith('-ivsk_m')).map((key)=>key.split('-')[0]);
|
|
73
75
|
return accounts.map((account)=>AztecAddress.fromString(account));
|
|
74
76
|
}
|
|
77
|
+
/** Checks whether an account is registered in the key store. */ async hasAccount(account) {
|
|
78
|
+
return !!await this.#keys.getAsync(`${account.toString()}-ivsk_m`);
|
|
79
|
+
}
|
|
75
80
|
/**
|
|
76
81
|
* Gets the key validation request for a given master public key hash and contract address.
|
|
77
82
|
* @throws If the account corresponding to the master public key hash does not exist in the key store.
|
|
78
83
|
* @param pkMHash - The master public key hash.
|
|
79
84
|
* @param contractAddress - The contract address to silo the secret key in the key validation request with.
|
|
80
85
|
* @returns The key validation request.
|
|
81
|
-
*/
|
|
82
|
-
|
|
83
|
-
|
|
84
|
-
|
|
85
|
-
|
|
86
|
-
|
|
87
|
-
|
|
88
|
-
|
|
89
|
-
|
|
90
|
-
|
|
91
|
-
|
|
92
|
-
|
|
93
|
-
|
|
94
|
-
|
|
95
|
-
|
|
96
|
-
|
|
97
|
-
|
|
98
|
-
|
|
99
|
-
|
|
100
|
-
|
|
101
|
-
|
|
102
|
-
|
|
103
|
-
|
|
104
|
-
|
|
105
|
-
|
|
106
|
-
|
|
107
|
-
|
|
86
|
+
*/ getKeyValidationRequest(pkMHash, contractAddress) {
|
|
87
|
+
return this.#db.transactionAsync(async ()=>{
|
|
88
|
+
const [keyPrefix, account] = await this.getKeyPrefixAndAccount(pkMHash);
|
|
89
|
+
// Load the stored master public key point. The returned KVR carries only the hash, but we
|
|
90
|
+
// use the point here as a witness for two integrity checks below: (1) it matches the supplied
|
|
91
|
+
// hash, and (2) it matches the value derived from the stored secret key.
|
|
92
|
+
const pkMBuffer = await this.#keys.getAsync(`${account.toString()}-${keyPrefix}pk_m`);
|
|
93
|
+
if (!pkMBuffer) {
|
|
94
|
+
throw new Error(`Could not find ${keyPrefix}pk_m for account ${account.toString()} whose address was successfully obtained with ${keyPrefix}pk_m_hash ${pkMHash.toString()}.`);
|
|
95
|
+
}
|
|
96
|
+
const pkM = Point.fromBuffer(pkMBuffer);
|
|
97
|
+
// Now we find the secret key for the public key
|
|
98
|
+
const skStorageSuffix = secretKeyStorageSuffix(keyPrefix);
|
|
99
|
+
const skMBuffer = await this.#keys.getAsync(`${account.toString()}-${skStorageSuffix}`);
|
|
100
|
+
if (!skMBuffer) {
|
|
101
|
+
throw new Error(`Could not find ${skStorageSuffix} for account ${account.toString()} whose address was successfully obtained with ${keyPrefix}pk_m_hash ${pkMHash.toString()}.`);
|
|
102
|
+
}
|
|
103
|
+
const skM = GrumpkinScalar.fromBuffer(skMBuffer);
|
|
104
|
+
// The remaining awaits are non-DB computations. They are safe because no further IDB operations follow them.
|
|
105
|
+
const computedPkMHash = await hashPublicKey(pkM);
|
|
106
|
+
if (!computedPkMHash.equals(pkMHash)) {
|
|
107
|
+
throw new Error(`Could not find ${keyPrefix}pkM for ${keyPrefix}pk_m_hash ${pkMHash.toString()}.`);
|
|
108
|
+
}
|
|
109
|
+
const derivedPkM = await derivePublicKeyFromSecretKey(skM);
|
|
110
|
+
if (!derivedPkM.equals(pkM)) {
|
|
111
|
+
throw new Error(`Could not derive ${keyPrefix}pkM from ${keyPrefix}skM.`);
|
|
112
|
+
}
|
|
113
|
+
const skApp = await computeAppSecretKey(skM, contractAddress, keyPrefix);
|
|
114
|
+
return new KeyValidationRequest(pkMHash, skApp);
|
|
115
|
+
});
|
|
108
116
|
}
|
|
109
117
|
/**
|
|
110
118
|
* Gets the master nullifier public key for a given account.
|
|
@@ -182,27 +190,52 @@ import { KEY_PREFIXES, computeAppSecretKey, deriveKeys, derivePublicKeyFromSecre
|
|
|
182
190
|
masterOutgoingViewingSecretKey.hi,
|
|
183
191
|
masterOutgoingViewingSecretKey.lo,
|
|
184
192
|
app
|
|
185
|
-
],
|
|
193
|
+
], DomainSeparator.OVSK_M);
|
|
186
194
|
}
|
|
187
195
|
/**
|
|
188
|
-
* Retrieves the sk_m corresponding to the pk_m.
|
|
189
|
-
* @throws If the provided
|
|
190
|
-
* @param
|
|
196
|
+
* Retrieves the sk_m corresponding to the given pk_m hash.
|
|
197
|
+
* @throws If the provided hash is not associated with any of the registered accounts.
|
|
198
|
+
* @param pkMHash - The master public key hash to get secret key for.
|
|
191
199
|
* @returns A Promise that resolves to sk_m.
|
|
192
200
|
* @dev Used when feeding the sk_m to the kernel circuit for keys verification.
|
|
193
|
-
*/
|
|
194
|
-
|
|
195
|
-
|
|
196
|
-
|
|
197
|
-
|
|
198
|
-
|
|
199
|
-
|
|
200
|
-
|
|
201
|
-
|
|
202
|
-
|
|
203
|
-
|
|
204
|
-
|
|
205
|
-
|
|
201
|
+
*/ getMasterSecretKey(pkMHash) {
|
|
202
|
+
return this.#db.transactionAsync(async ()=>{
|
|
203
|
+
const [keyPrefix, account] = await this.getKeyPrefixAndAccount(pkMHash);
|
|
204
|
+
const skStorageSuffix = secretKeyStorageSuffix(keyPrefix);
|
|
205
|
+
const secretKeyBuffer = await this.#keys.getAsync(`${account.toString()}-${skStorageSuffix}`);
|
|
206
|
+
if (!secretKeyBuffer) {
|
|
207
|
+
throw new Error(`Could not find ${skStorageSuffix} for ${keyPrefix}pk_m_hash ${pkMHash.toString()}. This should not happen.`);
|
|
208
|
+
}
|
|
209
|
+
const skM = GrumpkinScalar.fromBuffer(secretKeyBuffer);
|
|
210
|
+
// Non-DB computation — safe because no further IDB operations follow.
|
|
211
|
+
// Integrity check: confirm the stored secret key still derives the requested hash. The check
|
|
212
|
+
// is hash-based rather than point-equal because the on-disk identifier is `pk_m_hash`;
|
|
213
|
+
// cryptographic collision resistance of `hashPublicKey` makes this equivalent to a
|
|
214
|
+
// direct point comparison in practice.
|
|
215
|
+
const derivedPkM = await derivePublicKeyFromSecretKey(skM);
|
|
216
|
+
const derivedPkMHash = await hashPublicKey(derivedPkM);
|
|
217
|
+
if (!derivedPkMHash.equals(pkMHash)) {
|
|
218
|
+
throw new Error(`Could not find ${skStorageSuffix} for ${keyPrefix}pk_m_hash ${pkMHash.toString()} in secret keys buffer.`);
|
|
219
|
+
}
|
|
220
|
+
return skM;
|
|
221
|
+
});
|
|
222
|
+
}
|
|
223
|
+
/**
|
|
224
|
+
* Checks whether a given account has a key matching the provided master public key hash.
|
|
225
|
+
* @param account - The account address to check.
|
|
226
|
+
* @param pkMHash - The master public key hash to look for.
|
|
227
|
+
* @returns True if the account has a key with the given hash.
|
|
228
|
+
*/ accountHasKey(account, pkMHash) {
|
|
229
|
+
return this.#db.transactionAsync(async ()=>{
|
|
230
|
+
const pkMHashBuffer = serializeToBuffer(pkMHash);
|
|
231
|
+
for (const prefix of KEY_PREFIXES){
|
|
232
|
+
const stored = await this.#keys.getAsync(`${account.toString()}-${prefix}pk_m_hash`);
|
|
233
|
+
if (stored && Buffer.from(stored).equals(pkMHashBuffer)) {
|
|
234
|
+
return true;
|
|
235
|
+
}
|
|
236
|
+
}
|
|
237
|
+
return false;
|
|
238
|
+
});
|
|
206
239
|
}
|
|
207
240
|
/**
|
|
208
241
|
* Gets the key prefix and account address for a given value.
|
package/package.json
CHANGED
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "@aztec/key-store",
|
|
3
|
-
"version": "0.0.1-commit.
|
|
3
|
+
"version": "0.0.1-commit.e57c76e",
|
|
4
4
|
"type": "module",
|
|
5
5
|
"exports": "./dest/index.js",
|
|
6
6
|
"typedocOptions": {
|
|
@@ -57,10 +57,10 @@
|
|
|
57
57
|
]
|
|
58
58
|
},
|
|
59
59
|
"dependencies": {
|
|
60
|
-
"@aztec/constants": "0.0.1-commit.
|
|
61
|
-
"@aztec/foundation": "0.0.1-commit.
|
|
62
|
-
"@aztec/kv-store": "0.0.1-commit.
|
|
63
|
-
"@aztec/stdlib": "0.0.1-commit.
|
|
60
|
+
"@aztec/constants": "0.0.1-commit.e57c76e",
|
|
61
|
+
"@aztec/foundation": "0.0.1-commit.e57c76e",
|
|
62
|
+
"@aztec/kv-store": "0.0.1-commit.e57c76e",
|
|
63
|
+
"@aztec/stdlib": "0.0.1-commit.e57c76e",
|
|
64
64
|
"tslib": "^2.4.0"
|
|
65
65
|
},
|
|
66
66
|
"devDependencies": {
|
package/src/key_store.ts
CHANGED
|
@@ -1,4 +1,4 @@
|
|
|
1
|
-
import {
|
|
1
|
+
import { DomainSeparator } from '@aztec/constants';
|
|
2
2
|
import { poseidon2HashWithSeparator } from '@aztec/foundation/crypto/poseidon';
|
|
3
3
|
import { Fr } from '@aztec/foundation/curves/bn254';
|
|
4
4
|
import { GrumpkinScalar, Point } from '@aztec/foundation/curves/grumpkin';
|
|
@@ -15,6 +15,7 @@ import {
|
|
|
15
15
|
computeAppSecretKey,
|
|
16
16
|
deriveKeys,
|
|
17
17
|
derivePublicKeyFromSecretKey,
|
|
18
|
+
hashPublicKey,
|
|
18
19
|
} from '@aztec/stdlib/keys';
|
|
19
20
|
|
|
20
21
|
/** Maps a key prefix to the storage suffix for the corresponding master secret key. */
|
|
@@ -57,17 +58,24 @@ export class KeyStore {
|
|
|
57
58
|
masterIncomingViewingSecretKey,
|
|
58
59
|
masterOutgoingViewingSecretKey,
|
|
59
60
|
masterTaggingSecretKey,
|
|
61
|
+
masterNullifierPublicKey,
|
|
62
|
+
masterOutgoingViewingPublicKey,
|
|
63
|
+
masterTaggingPublicKey,
|
|
60
64
|
publicKeys,
|
|
61
65
|
} = await deriveKeys(sk);
|
|
62
66
|
|
|
63
|
-
const completeAddress = await CompleteAddress.
|
|
67
|
+
const completeAddress = await CompleteAddress.fromPublicKeysAndPartialAddress(publicKeys, partialAddress);
|
|
64
68
|
const { address: account } = completeAddress;
|
|
65
69
|
|
|
66
|
-
//
|
|
67
|
-
|
|
68
|
-
|
|
69
|
-
|
|
70
|
-
|
|
70
|
+
// The kernel cannot check that nhpk/ovpk/tpk are on-curve or non-infinity, so the PXE/key-store
|
|
71
|
+
// must guarantee it before persistence. By design, the above derivation produces points that are on
|
|
72
|
+
// the curve and not at infinity.
|
|
73
|
+
|
|
74
|
+
// The npk/ovpk/tpk hashes are already in publicKeys; ivpk_m_hash is computed for indexing.
|
|
75
|
+
const masterIncomingViewingPublicKeyHash = await hashPublicKey(publicKeys.ivpkM);
|
|
76
|
+
|
|
77
|
+
// The Message Signing and Fallback Keys don't have a derivation path yet, so we just use the default values for their hashes.
|
|
78
|
+
// So we avoid storing them persistently. The default hash is still required for address derivation
|
|
71
79
|
|
|
72
80
|
await this.#db.transactionAsync(async () => {
|
|
73
81
|
// Naming of keys is as follows ${account}-${n/iv/ov/t}${sk/pk}_m
|
|
@@ -76,17 +84,17 @@ export class KeyStore {
|
|
|
76
84
|
await this.#keys.set(`${account.toString()}-tsk_m`, masterTaggingSecretKey.toBuffer());
|
|
77
85
|
await this.#keys.set(`${account.toString()}-nhk_m`, masterNullifierHidingKey.toBuffer());
|
|
78
86
|
|
|
79
|
-
await this.#keys.set(`${account.toString()}-npk_m`,
|
|
80
|
-
await this.#keys.set(`${account.toString()}-ivpk_m`, publicKeys.
|
|
81
|
-
await this.#keys.set(`${account.toString()}-ovpk_m`,
|
|
82
|
-
await this.#keys.set(`${account.toString()}-tpk_m`,
|
|
87
|
+
await this.#keys.set(`${account.toString()}-npk_m`, masterNullifierPublicKey.toBuffer());
|
|
88
|
+
await this.#keys.set(`${account.toString()}-ivpk_m`, publicKeys.ivpkM.toBuffer());
|
|
89
|
+
await this.#keys.set(`${account.toString()}-ovpk_m`, masterOutgoingViewingPublicKey.toBuffer());
|
|
90
|
+
await this.#keys.set(`${account.toString()}-tpk_m`, masterTaggingPublicKey.toBuffer());
|
|
83
91
|
|
|
84
92
|
// We store pk_m_hash under `account-{n/iv/ov/t}pk_m_hash` key to be able to obtain address and key prefix
|
|
85
93
|
// using the #getKeyPrefixAndAccount function later on
|
|
86
|
-
await this.#keys.set(`${account.toString()}-npk_m_hash`,
|
|
94
|
+
await this.#keys.set(`${account.toString()}-npk_m_hash`, publicKeys.npkMHash.toBuffer());
|
|
87
95
|
await this.#keys.set(`${account.toString()}-ivpk_m_hash`, masterIncomingViewingPublicKeyHash.toBuffer());
|
|
88
|
-
await this.#keys.set(`${account.toString()}-ovpk_m_hash`,
|
|
89
|
-
await this.#keys.set(`${account.toString()}-tpk_m_hash`,
|
|
96
|
+
await this.#keys.set(`${account.toString()}-ovpk_m_hash`, publicKeys.ovpkMHash.toBuffer());
|
|
97
|
+
await this.#keys.set(`${account.toString()}-tpk_m_hash`, publicKeys.tpkMHash.toBuffer());
|
|
90
98
|
});
|
|
91
99
|
|
|
92
100
|
// At last, we return the newly derived account address
|
|
@@ -104,6 +112,11 @@ export class KeyStore {
|
|
|
104
112
|
return accounts.map(account => AztecAddress.fromString(account));
|
|
105
113
|
}
|
|
106
114
|
|
|
115
|
+
/** Checks whether an account is registered in the key store. */
|
|
116
|
+
public async hasAccount(account: AztecAddress): Promise<boolean> {
|
|
117
|
+
return !!(await this.#keys.getAsync(`${account.toString()}-ivsk_m`));
|
|
118
|
+
}
|
|
119
|
+
|
|
107
120
|
/**
|
|
108
121
|
* Gets the key validation request for a given master public key hash and contract address.
|
|
109
122
|
* @throws If the account corresponding to the master public key hash does not exist in the key store.
|
|
@@ -111,44 +124,48 @@ export class KeyStore {
|
|
|
111
124
|
* @param contractAddress - The contract address to silo the secret key in the key validation request with.
|
|
112
125
|
* @returns The key validation request.
|
|
113
126
|
*/
|
|
114
|
-
public
|
|
115
|
-
|
|
127
|
+
public getKeyValidationRequest(pkMHash: Fr, contractAddress: AztecAddress): Promise<KeyValidationRequest> {
|
|
128
|
+
return this.#db.transactionAsync(async () => {
|
|
129
|
+
const [keyPrefix, account] = await this.getKeyPrefixAndAccount(pkMHash);
|
|
130
|
+
|
|
131
|
+
// Load the stored master public key point. The returned KVR carries only the hash, but we
|
|
132
|
+
// use the point here as a witness for two integrity checks below: (1) it matches the supplied
|
|
133
|
+
// hash, and (2) it matches the value derived from the stored secret key.
|
|
134
|
+
const pkMBuffer = await this.#keys.getAsync(`${account.toString()}-${keyPrefix}pk_m`);
|
|
135
|
+
if (!pkMBuffer) {
|
|
136
|
+
throw new Error(
|
|
137
|
+
`Could not find ${keyPrefix}pk_m for account ${account.toString()} whose address was successfully obtained with ${keyPrefix}pk_m_hash ${pkMHash.toString()}.`,
|
|
138
|
+
);
|
|
139
|
+
}
|
|
116
140
|
|
|
117
|
-
|
|
118
|
-
const pkMBuffer = await this.#keys.getAsync(`${account.toString()}-${keyPrefix}pk_m`);
|
|
119
|
-
if (!pkMBuffer) {
|
|
120
|
-
throw new Error(
|
|
121
|
-
`Could not find ${keyPrefix}pk_m for account ${account.toString()} whose address was successfully obtained with ${keyPrefix}pk_m_hash ${pkMHash.toString()}.`,
|
|
122
|
-
);
|
|
123
|
-
}
|
|
141
|
+
const pkM = Point.fromBuffer(pkMBuffer);
|
|
124
142
|
|
|
125
|
-
|
|
126
|
-
|
|
127
|
-
|
|
128
|
-
|
|
129
|
-
|
|
143
|
+
// Now we find the secret key for the public key
|
|
144
|
+
const skStorageSuffix = secretKeyStorageSuffix(keyPrefix);
|
|
145
|
+
const skMBuffer = await this.#keys.getAsync(`${account.toString()}-${skStorageSuffix}`);
|
|
146
|
+
if (!skMBuffer) {
|
|
147
|
+
throw new Error(
|
|
148
|
+
`Could not find ${skStorageSuffix} for account ${account.toString()} whose address was successfully obtained with ${keyPrefix}pk_m_hash ${pkMHash.toString()}.`,
|
|
149
|
+
);
|
|
150
|
+
}
|
|
130
151
|
|
|
131
|
-
|
|
132
|
-
const skStorageSuffix = secretKeyStorageSuffix(keyPrefix);
|
|
133
|
-
const skMBuffer = await this.#keys.getAsync(`${account.toString()}-${skStorageSuffix}`);
|
|
134
|
-
if (!skMBuffer) {
|
|
135
|
-
throw new Error(
|
|
136
|
-
`Could not find ${skStorageSuffix} for account ${account.toString()} whose address was successfully obtained with ${keyPrefix}pk_m_hash ${pkMHash.toString()}.`,
|
|
137
|
-
);
|
|
138
|
-
}
|
|
152
|
+
const skM = GrumpkinScalar.fromBuffer(skMBuffer);
|
|
139
153
|
|
|
140
|
-
|
|
154
|
+
// The remaining awaits are non-DB computations. They are safe because no further IDB operations follow them.
|
|
155
|
+
const computedPkMHash = await hashPublicKey(pkM);
|
|
156
|
+
if (!computedPkMHash.equals(pkMHash)) {
|
|
157
|
+
throw new Error(`Could not find ${keyPrefix}pkM for ${keyPrefix}pk_m_hash ${pkMHash.toString()}.`);
|
|
158
|
+
}
|
|
141
159
|
|
|
142
|
-
|
|
143
|
-
|
|
144
|
-
|
|
145
|
-
|
|
146
|
-
}
|
|
160
|
+
const derivedPkM = await derivePublicKeyFromSecretKey(skM);
|
|
161
|
+
if (!derivedPkM.equals(pkM)) {
|
|
162
|
+
throw new Error(`Could not derive ${keyPrefix}pkM from ${keyPrefix}skM.`);
|
|
163
|
+
}
|
|
147
164
|
|
|
148
|
-
|
|
149
|
-
const skApp = await computeAppSecretKey(skM, contractAddress, keyPrefix!);
|
|
165
|
+
const skApp = await computeAppSecretKey(skM, contractAddress, keyPrefix!);
|
|
150
166
|
|
|
151
|
-
|
|
167
|
+
return new KeyValidationRequest(pkMHash, skApp);
|
|
168
|
+
});
|
|
152
169
|
}
|
|
153
170
|
|
|
154
171
|
/**
|
|
@@ -249,35 +266,65 @@ export class KeyStore {
|
|
|
249
266
|
|
|
250
267
|
return poseidon2HashWithSeparator(
|
|
251
268
|
[masterOutgoingViewingSecretKey.hi, masterOutgoingViewingSecretKey.lo, app],
|
|
252
|
-
|
|
269
|
+
DomainSeparator.OVSK_M,
|
|
253
270
|
);
|
|
254
271
|
}
|
|
255
272
|
|
|
256
273
|
/**
|
|
257
|
-
* Retrieves the sk_m corresponding to the pk_m.
|
|
258
|
-
* @throws If the provided
|
|
259
|
-
* @param
|
|
274
|
+
* Retrieves the sk_m corresponding to the given pk_m hash.
|
|
275
|
+
* @throws If the provided hash is not associated with any of the registered accounts.
|
|
276
|
+
* @param pkMHash - The master public key hash to get secret key for.
|
|
260
277
|
* @returns A Promise that resolves to sk_m.
|
|
261
278
|
* @dev Used when feeding the sk_m to the kernel circuit for keys verification.
|
|
262
279
|
*/
|
|
263
|
-
public
|
|
264
|
-
|
|
280
|
+
public getMasterSecretKey(pkMHash: Fr): Promise<GrumpkinScalar> {
|
|
281
|
+
return this.#db.transactionAsync(async () => {
|
|
282
|
+
const [keyPrefix, account] = await this.getKeyPrefixAndAccount(pkMHash);
|
|
283
|
+
|
|
284
|
+
const skStorageSuffix = secretKeyStorageSuffix(keyPrefix);
|
|
285
|
+
const secretKeyBuffer = await this.#keys.getAsync(`${account.toString()}-${skStorageSuffix}`);
|
|
286
|
+
if (!secretKeyBuffer) {
|
|
287
|
+
throw new Error(
|
|
288
|
+
`Could not find ${skStorageSuffix} for ${keyPrefix}pk_m_hash ${pkMHash.toString()}. This should not happen.`,
|
|
289
|
+
);
|
|
290
|
+
}
|
|
265
291
|
|
|
266
|
-
|
|
267
|
-
|
|
268
|
-
|
|
269
|
-
|
|
270
|
-
|
|
271
|
-
|
|
272
|
-
|
|
292
|
+
const skM = GrumpkinScalar.fromBuffer(secretKeyBuffer);
|
|
293
|
+
|
|
294
|
+
// Non-DB computation — safe because no further IDB operations follow.
|
|
295
|
+
// Integrity check: confirm the stored secret key still derives the requested hash. The check
|
|
296
|
+
// is hash-based rather than point-equal because the on-disk identifier is `pk_m_hash`;
|
|
297
|
+
// cryptographic collision resistance of `hashPublicKey` makes this equivalent to a
|
|
298
|
+
// direct point comparison in practice.
|
|
299
|
+
const derivedPkM = await derivePublicKeyFromSecretKey(skM);
|
|
300
|
+
const derivedPkMHash = await hashPublicKey(derivedPkM);
|
|
301
|
+
if (!derivedPkMHash.equals(pkMHash)) {
|
|
302
|
+
throw new Error(
|
|
303
|
+
`Could not find ${skStorageSuffix} for ${keyPrefix}pk_m_hash ${pkMHash.toString()} in secret keys buffer.`,
|
|
304
|
+
);
|
|
305
|
+
}
|
|
273
306
|
|
|
274
|
-
|
|
275
|
-
|
|
276
|
-
|
|
277
|
-
throw new Error(`Could not find ${skStorageSuffix} for ${keyPrefix}pkM ${pkM.toString()} in secret keys buffer.`);
|
|
278
|
-
}
|
|
307
|
+
return skM;
|
|
308
|
+
});
|
|
309
|
+
}
|
|
279
310
|
|
|
280
|
-
|
|
311
|
+
/**
|
|
312
|
+
* Checks whether a given account has a key matching the provided master public key hash.
|
|
313
|
+
* @param account - The account address to check.
|
|
314
|
+
* @param pkMHash - The master public key hash to look for.
|
|
315
|
+
* @returns True if the account has a key with the given hash.
|
|
316
|
+
*/
|
|
317
|
+
public accountHasKey(account: AztecAddress, pkMHash: Fr): Promise<boolean> {
|
|
318
|
+
return this.#db.transactionAsync(async () => {
|
|
319
|
+
const pkMHashBuffer = serializeToBuffer(pkMHash);
|
|
320
|
+
for (const prefix of KEY_PREFIXES) {
|
|
321
|
+
const stored = await this.#keys.getAsync(`${account.toString()}-${prefix}pk_m_hash`);
|
|
322
|
+
if (stored && Buffer.from(stored).equals(pkMHashBuffer)) {
|
|
323
|
+
return true;
|
|
324
|
+
}
|
|
325
|
+
}
|
|
326
|
+
return false;
|
|
327
|
+
});
|
|
281
328
|
}
|
|
282
329
|
|
|
283
330
|
/**
|