Blocks validity api #16
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Replace env::var with common::get_env_var for consistent error handling - Add skip_proof_verification CLI option with default false - Remove dotenv direct loading since it's handled by get_env_var - Improve code formatting and error handling in main() - Add proper error propagation with Result return type - Clean up imports and code organization
- feat(batch-processor): persist .db file locally and to IPFS regardless of batch completion - feat(state-proof-api): add block inclusion proof request handling - fix: resolve misc bugs in provider implementation
- Modified blocks-validity method to extract base_fee_per_gas from each block header - Return vector of base fees instead of boolean value - Handle None cases with default empty string
BREAKING CHANGES: - Merge blocks-validity and fee extraction into single RISC Zero method - Rename blocks-validity to validate_blocks_and_extract_fees - Replace verify_blocks.rs with extract_fees.rs binary Changes: - crates/methods/: * Remove blocks-validity crate * Add validate_blocks_and_extract_fees crate * Update Cargo.toml dependencies - crates/publisher/: * Remove verify_blocks.rs binary * Add extract_fees.rs binary * Update operations.rs to use new validation method * Update validator.rs with combined validation logic * Update lib.rs exports * Update Cargo.toml dependencies - crates/state-proof-api/: * Update main.rs to use extract_fees operation * Update API response handling This change simplifies the codebase by combining block validation and fee extraction into a single RISC Zero method, reducing duplication and improving maintainability.
- Add binary declaration in Cargo.toml for fetch-fees-proof - Add reqwest dependency for HTTP client functionality - Configure binary path to bin/fetch-fees-proof.rs
Add a test binary that fetches and verifies Stark proofs from the state-proof-api service. This binary: - Fetches block fee proofs for a specified block range - Deserializes the Stark proofs from bincode format - Decodes and displays the fee values from the proof journal - Verifies the Stark proof against the known image ID - Uses tracing for structured logging This is a testing utility to validate the state-proof-api service's proof generation and verification functionality.
- fix: resolve IPFS upload issue for incomplete batches - refactor: remove optional IPFS CID parameter from Rust and Starknet contracts - test: verify state-proof-api handles multi-batch requests correctly - feat: enhance fetch-fees-proof to consolidate fees from multiple Stark proofs
- Add IPFS download functionality to fee-proof API - Implement MMR state validation with IPFS hash verification - Add batch file handling with local fallback - Add proper error handling for IPFS operations - Fix unused error variable warning in validator.rs - Refactor state-proof-api response handling - Clean up imports and use proper Axum types - Improve error responses with proper content types - Add proper state management with Arc<AppState> - Fix handler signature and response types
- Add explicit use statements for required dependencies - Remove debug println statements from validator code - Clean up tracing log levels and messages - Remove redundant debug logging in MMR root verification
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.