Skip to content

Commit

Permalink
fix: missing NoteSelector from JSON RPC proxies (#7493)
Browse files Browse the repository at this point in the history
  • Loading branch information
spypsy authored Jul 16, 2024
1 parent 0599500 commit b209fad
Show file tree
Hide file tree
Showing 4 changed files with 24 additions and 7 deletions.
6 changes: 4 additions & 2 deletions yarn-project/aztec-node/src/aztec-node/http_rpc_server.ts
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@ import {
UnencryptedL2BlockL2Logs,
} from '@aztec/circuit-types';
import { FunctionSelector, Header } from '@aztec/circuits.js';
import { NoteSelector } from '@aztec/foundation/abi';
import { AztecAddress } from '@aztec/foundation/aztec-address';
import { EthAddress } from '@aztec/foundation/eth-address';
import { Fr } from '@aztec/foundation/fields';
Expand Down Expand Up @@ -43,12 +44,13 @@ export function createAztecNodeRpcServer(node: AztecNode) {
SiblingPath,
},
{
EncryptedNoteL2BlockL2Logs,
NoteSelector,
NullifierMembershipWitness,
PublicSimulationOutput,
Tx,
TxReceipt,
EncryptedNoteL2BlockL2Logs,
UnencryptedL2BlockL2Logs,
NullifierMembershipWitness,
},
// disable methods not part of the AztecNode interface
['start', 'stop'],
Expand Down
9 changes: 8 additions & 1 deletion yarn-project/aztec.js/src/rpc_clients/pxe_client.ts
Original file line number Diff line number Diff line change
Expand Up @@ -54,9 +54,16 @@ export const createPXEClient = (url: string, fetch = makeFetch([1, 2, 3], false)
Point,
TxExecutionRequest,
TxHash,
},
{
EncryptedNoteL2BlockL2Logs,
NoteSelector,
NullifierMembershipWitness,
SimulatedTx,
Tx,
TxReceipt,
UnencryptedL2BlockL2Logs,
},
{ Tx, SimulatedTx, TxReceipt, EncryptedNoteL2BlockL2Logs, UnencryptedL2BlockL2Logs, NullifierMembershipWitness },
false,
'pxe',
fetch,
Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import { FunctionSelector, Header } from '@aztec/circuits.js';
import { EventSelector } from '@aztec/foundation/abi';
import { EventSelector, NoteSelector } from '@aztec/foundation/abi';
import { AztecAddress } from '@aztec/foundation/aztec-address';
import { EthAddress } from '@aztec/foundation/eth-address';
import { Fr } from '@aztec/foundation/fields';
Expand Down Expand Up @@ -44,12 +44,13 @@ export function createAztecNodeClient(url: string, fetch = defaultFetch): AztecN
SiblingPath,
},
{
EncryptedNoteL2BlockL2Logs,
NoteSelector,
NullifierMembershipWitness,
PublicSimulationOutput,
Tx,
TxReceipt,
EncryptedNoteL2BlockL2Logs,
UnencryptedL2BlockL2Logs,
NullifierMembershipWitness,
},
false,
'node',
Expand Down
9 changes: 8 additions & 1 deletion yarn-project/pxe/src/pxe_http/pxe_http_server.ts
Original file line number Diff line number Diff line change
Expand Up @@ -50,9 +50,16 @@ export function createPXERpcServer(pxeService: PXE): JsonRpcServer {
L2Block,
TxEffect,
LogId,
},
{
EncryptedNoteL2BlockL2Logs,
NoteSelector,
NullifierMembershipWitness,
SimulatedTx,
Tx,
TxReceipt,
UnencryptedL2BlockL2Logs,
},
{ SimulatedTx, Tx, TxReceipt, EncryptedNoteL2BlockL2Logs, UnencryptedL2BlockL2Logs, NullifierMembershipWitness },
['start', 'stop'],
);
}
Expand Down

0 comments on commit b209fad

Please sign in to comment.