This is updated `RPC` implementation according to the new structure we had decided by Sansh2356 · Pull Request #224 · braidpool/braidpool · GitHub
Skip to content

This is updated RPC implementation according to the new structure we had decided #224

Merged
mcelrath merged 3 commits into
devfrom
feat-rpc
Jul 12, 2025
Merged

This is updated RPC implementation according to the new structure we had decided #224
mcelrath merged 3 commits into
devfrom
feat-rpc

Conversation

@Sansh2356

Copy link
Copy Markdown
Contributor
  • It includes rpc methods accessible both via cli as well as from frontend via curl or axios.
  • Unit tests regarding each method is added respectively .
  • This PR also involves the replacing of snippets which are common and also the module accessibility structure as well.

@mcelrath mcelrath left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good!

(Still need to test it)

Comment thread node/src/braid/tests.rs Outdated
Comment thread node/src/rpc_server.rs Outdated
Comment thread node/src/rpc_server.rs Outdated
@Sansh2356 Sansh2356 mentioned this pull request Jun 25, 2025
37 tasks
@Sansh2356 Sansh2356 added the enhancement New feature or request label Jul 10, 2025

@mcelrath mcelrath left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested this with curl, seems fine. We need data!

Just fix the logic in Braid::extend() otherwise this looks good.

Comment thread node/src/bead/tests.rs
Comment thread node/src/braid/mod.rs Outdated
Comment thread node/src/braid/mod.rs Outdated
Comment thread node/src/rpc_server.rs
Comment thread node/src/main.rs
Removing commented fields from test.rs

cargo-fmt
@mcelrath mcelrath merged commit edf46f3 into dev Jul 12, 2025
11 checks passed
@mcelrath mcelrath deleted the feat-rpc branch July 12, 2025 17:18
@Sansh2356 Sansh2356 changed the title This this is updated RPC implementation according to the new structure we had decided This is updated RPC implementation according to the new structure we had decided Jul 13, 2025
@Sansh2356 Sansh2356 self-assigned this Oct 14, 2025
mcelrath pushed a commit to mcelrath/braidpool that referenced this pull request Nov 18, 2025
…ure we had decided (braidpool#224)

* feat:rpc addition along with refactoring of cli

* Updating the config.rs with rpc_config

* modifying the braid initialization && reverting extend

Removing commented fields from test.rs

cargo-fmt
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants