Closed namn-grg closed 4 weeks ago
@coderabbitai review
Ahoy there! The changes to the bolt-sidecar
project bring forth new configuration variables for commit boosting, refine the management of dependencies, and enhance the signing functionalities. The CommitBoostClient
now sails with a SignerClient
for handling requests, while the configuration structures have been updated to embrace new JWT settings. The code's overall structure has been streamlined for better clarity and maintainability, savvy?
Files | Change Summary |
---|---|
bolt-sidecar/.env.example |
Added BOLT_SIDECAR_COMMIT_BOOST_URL and BOLT_SIDECAR_COMMIT_BOOST_JWT_HEX for commit boosting. |
bolt-sidecar/Cargo.toml |
Updated cb-common dependency with a specific revision; removed cb-crypto . |
bolt-sidecar/src/client/commit_boost.rs |
Refactored to use CommitBoostSigner with a focus on signing operations. |
bolt-sidecar/src/config/mod.rs |
Renamed commit_boost_url to commit_boost_address and added commit_boost_jwt_hex . |
bolt-sidecar/src/config/signing.rs |
Introduced commit_boost_jwt_hex field to SigningOpts structure. |
bolt-sidecar/src/crypto/bls.rs |
Updated signing methods to use fixed-size arrays for better performance. |
bolt-sidecar/src/driver.rs |
Simplified SidecarDriver struct; updated signing process to accommodate new signer types. |
bolt-sidecar/src/lib.rs |
Added CommitBoostSigner to the public interface. |
bolt-sidecar/src/primitives/commitment.rs |
Modified commit_and_sign to require SignerECDSAAsync and updated error handling. |
bolt-sidecar/src/test_util.rs |
Changed TestSignableData to use fixed-size arrays for data handling. |
sequenceDiagram
participant Client
participant CommitBoostSigner
participant SignerClient
Client->>CommitBoostSigner: Request to sign data
CommitBoostSigner->>SignerClient: Forward signing request
SignerClient-->>CommitBoostSigner: Return signed data
CommitBoostSigner-->>Client: Send signed data
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
@coderabbitai review
Just working on a mock test
partially solves #201
Note - This is not e2e tested with commit-boost
Summary by CodeRabbit
New Features
CommitBoostSigner
, to the public interface.Improvements
SidecarDriver
to streamline the signing process and support asynchronous operations.Bug Fixes
commit_and_sign
method.Chores