-
Notifications
You must be signed in to change notification settings - Fork 2
BFT snapshots #3
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.
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
Open
lploom
wants to merge
37
commits into
main
Choose a base branch
from
bft-snapshots
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
37 commits
Select commit
Hold shift + click to select a range
ca1bdfd
Rename miner address to payload root
lploom 4130e90
BFT snapshots
lploom b39f6a4
fix import
lploom 4529b97
fix genesis block hashes
lploom 8336ef4
Update fuzzer for new block structure
lploom 8623efd
Update getblocktemplate and submitblock for new block structure
lploom e6cbe81
Track last UTB in block index
lploom 66c8423
Add CBlockHeader::MAX_PAYLOAD_SIZE
lploom 0c2cd0f
Improve trust base verification
lploom c6501af
Cross-platform random bytes
lploom ec992bf
Add trust base total stake overflow checks
lploom ba42a92
Truncate external error message before logging
lploom 3ca823c
Add MAX_BFT_RESPONSE_SIZE
lploom 1808636
Refactor bft client
lploom 8459b36
Optimize CBlockHeader serialization
lploom eb1d706
Refactor HttpBFTClient to use Pimpl for httplib::Client
lploom c11d6b0
Use secp256k1_context_static for signature verification
lploom d4502a0
Restrict permissions of miner_state.json to owner-only
lploom 7fc7036
More reliable initialization in HttpBFTClient
lploom 991e10c
Fix CBlockIndex::ToString
lploom 0d1e803
Remove unused imports
lploom 582deec
Remove a redundant deep copy
lploom ebd5485
Remove redundant field
lploom 446f267
Consolidate error handling
lploom cd3a256
Add trust base validation to CheckBlockHeader
lploom 075c64a
Wrap miner worker thread in try-catch
lploom 101df88
Reorder RPCServer params to match the declaration
lploom 4776ad9
Use same version in HandleGetBlockTemplate and miner
lploom b502252
Extract CBOR tag prepending into a helper function
lploom 8cbd497
Use correct epoch if trust base parsing fails
lploom c971e00
Update submitheader for new block structure
lploom a42d9f9
Merge branch 'main' into bft-snapshots
lploom ee14107
Fix submitheader payload handling and testnet functional tests
lploom f7d9fdb
Mirror request id in rpc response
lploom fdc8e3a
Update trust base cbor tags
lploom 9371d1c
Update trust base cbor tags
lploom fc15973
Minor: wording improvements in the block payload format spec
ahtotruu File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,3 +1,6 @@ | ||
| # Track prebuilt test chains via Git LFS | ||
|
|
||
| test/functional/test_chains/** filter=lfs diff=lfs merge=lfs -text | ||
|
|
||
| # keep bash scripts in LF | ||
| *.sh text eol=lf |
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The specified git tag
v0.38.0forcpp-httplibdoes not appear to exist in the officialyhirose/cpp-httplibrepository. The latest version seems to follow av0.major.minorscheme (e.g.,v0.15.3). This could be a typo and might cause build failures for users who do not have this specific commit cached. Please verify the tag and use a valid, existing tag to ensure build reproducibility.