Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Nym 2.0 #35

Merged
merged 49 commits into from
Jul 18, 2024
Merged

Nym 2.0 #35

merged 49 commits into from
Jul 18, 2024

Conversation

idky137
Copy link
Contributor

@idky137 idky137 commented Jun 20, 2024 •
edited
Loading

Refactor of the Nym code implementation in Zingo-Proxy. These changes are required to enable mutiple request types (RPCs) to be sent and received over the mixnet and to allow integration with Zingo-Proxy's request queue [WIP]. The walletside code is also refactored to ease wallet integration.

  • Adds walletrpc, production ready walletside nym code implemented on librustzcash's CompactTxStreamerClient.
  • Refactor NymServer, both to enable multiple request types and to enable integration with Zingo-Proxy's request queue.
  • Implement get_info lightwallet RPC over nym, along with non streaming RPCs required to obfuscate client syncing.

Closes #14, #17, #18

idky137 added 30 commits May 17, 2024 02:58
@idky137
added jsonrpc.rs
797218b
@idky137
added doc comments
0514f54
@idky137
jsonrpc builds
4605127
@idky137
move jsonrpc into modules
35bd4c7
@idky137
renamed HexSerializedBlock to ProxySerializedBlock
0d3075e
@idky137
get_lightd_info implemented - working on testnet, not in tests
8bc6702
@idky137
get_lightd_info implemented - working on testnet, not in tests
cd37ea6
@idky137
connects to zebra or zcash, doesnt run unless one is running.. fix this
ee402a0
@idky137
added startup connection checker
930b985
@idky137
removed commented code
261d638
@idky137
added welcome image
a4d24df
@idky137
added welcome image
b9185f1
@idky137
small fixes
336040a
@idky137
small fixes
be59ed8
@idky137
implemented send_transaction and get_best_block
31998da
@idky137
implemented get_taddress_txids
551c4ba
@idky137
started implementing get_block_range - parser required first
dfdd17c
@idky137
updated todos
cfa1d21
@idky137
removed extern crate
b1b09d3
@idky137
fixed merge conflicts
9c73ccd
@idky137
started adding block parser
f4d6e28
@idky137
added block and transaction structs
af5cc8a
@idky137
added comment to show untested code
c070337
@idky137
added BuildInfo struct and updated get_build_info
d56984e
@idky137
added gRPC server health check
bd0c4c0
@idky137
added parse_from_slice
fd3257b
@idky137
added block_height and block_hash getter logic
c6a3174
@idky137
builds, get_block_range not returning correct response
77f057d
@idky137
builds, compact block returned inccorrect
b72ecc6
@idky137
fixed txid conversion, get_block_range no works
f78ac42
@idky137
Copy link
Contributor Author

idky137 commented Jun 20, 2024

Built on top of add_jsonrpc [#26]. Commits prior to this comment are from that PR.

@idky137 idky137 marked this pull request as ready for review July 17, 2024 14:14
@Oscar-Pepper Oscar-Pepper merged commit 13c2f12 into zingolabs:dev Jul 18, 2024
5 of 12 checks passed
This was referenced Aug 16, 2024
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Oscar-Pepper Oscar-Pepper Oscar-Pepper approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Implement wallet-side Nym service RPCs on CompactTxStreamerClient and expose Nym Address
2 participants
@idky137 @Oscar-Pepper