Ipfs functionality #15
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
- Create new ipfs-utils crate for IPFS file operations - Implement IpfsManager for database file upload and download - Add comprehensive test suite including edge cases: - Basic upload and fetch functionality - Empty file handling - Invalid hash handling - Non-existent file handling - Invalid path handling
Add new ipfs-utils crate for handling MMR database files through IPFS. Features include: - Basic IPFS file upload and download functionality - File size limits and validation - Atomic file writes using temporary files - Error handling with custom error types - Basic hash validation - Comprehensive test suite Key functionality: - Upload .db files to IPFS and get CID - Fetch .db files from IPFS using hash - Connection checking and error handling - Configurable file size limits (default 50MB) Tests cover: - Basic upload/fetch operations - Empty file handling - Invalid hash handling - Invalid path handling - Nonexistent file handling
- Add new ipfs-utils crate for IPFS operations - Integrate IPFS storage in BatchProcessor - Upload MMR state to IPFS after each update - Add proper error handling for IPFS operations - Clean up temporary files after batch completion Key changes: - Upload MMR state to IPFS after every update, not just on completion - Use temporary local files during processing - Add StorageError variant for IPFS-related errors - Add IPFS manager to BatchProcessor - Convert paths properly between String and PathBuf
- Modified deploy-ethereum.sh to update both environment files in docker mode - Modified deploy-starknet.sh to update both environment files in docker mode - Added ENV_FILES array to handle multiple environment files - Added verification steps for environment file updates - Maintained backward compatibility for non-docker environments
- Replace felt!(hash) with Felt::from_hex() for IPFS hash conversion - Remove unused felt macro import - Use existing FeltConversion error handler
- Replace felt!(hash) with Felt::from_hex() for IPFS hash conversion in starknet-handler - Remove unused felt macro import - Use existing FeltConversion error handler - Add IPFS hash to BatchResult in batch processor - Update accumulator to handle IPFS hash storage
ametel01
force-pushed
the
ipfs-functionality
branch
from
3efccf5 to
cff8cda
Compare
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.