Skip to content
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

GetReceipts (0x0f) #386

Closed
Tracked by #128
mpaulucci opened this issue Sep 12, 2024 · 0 comments · Fixed by #885 or #1432
Closed
Tracked by #128

GetReceipts (0x0f) #386

mpaulucci opened this issue Sep 12, 2024 · 0 comments · Fixed by #885 or #1432
Labels
network Issues related to network communication

Comments

@mpaulucci
Copy link
Collaborator

No description provided.

@mpaulucci mpaulucci added this to the 2 - P2P Network milestone Sep 12, 2024
@mpaulucci mpaulucci added the network Issues related to network communication label Sep 12, 2024
@belfortep belfortep self-assigned this Oct 10, 2024
github-merge-queue bot pushed a commit that referenced this issue Oct 23, 2024
**Motivation**
Revival of #818

Adds the GetReceipts request and Receipts response for the eth protocol

**Description**
- Adds GetReceipts and Receipts with encode/decode capabilities
- rustic test with UDP sockets sending the data
- Implements get_all_receipts_by_hash for storage

Closes #386
Closes #387
@github-project-automation github-project-automation bot moved this from In Progress to Done in ethrex_l1 Oct 23, 2024
ElFantasma pushed a commit that referenced this issue Oct 23, 2024
**Motivation**
Revival of #818

Adds the GetReceipts request and Receipts response for the eth protocol

**Description**
- Adds GetReceipts and Receipts with encode/decode capabilities
- rustic test with UDP sockets sending the data
- Implements get_all_receipts_by_hash for storage

Closes #386
Closes #387
@fmoletta fmoletta reopened this Nov 22, 2024
github-merge-queue bot pushed a commit that referenced this issue Dec 10, 2024
**Motivation**
Support snap-sync
<!-- Why does this pull request exist? What are its goals? -->

**Description**
Implements the snap-sync cycle with the following caveats:

- No state healing is implemented (We asume all the state is available)
- Storage ranges are fetched in full (We don't partition storage range
requests, if an account's storage wasn't fully fetched as part of a
batch it is requested again)
- Minimum parallelization (We are currenltly testing against a single
node so not all parallelizable processes are being parallelized.
- Receipts are not fetched (waiting on #386)
- State is not blocked during sync (this could lead to the wrong data
being sent by rpc and p2p during an active sync)

Other changes:
- Move `crates/networking/p2p/README.rs` to
`crates/networking/docs/Network.md`
- Add doc for Sync in `crates/networking/docs/Sync`
- Reduce size of `RLPxError`


<!-- A clear and concise general description of the changes this PR
introduces -->

<!-- Link to issues: Resolves #111, Resolves #222 -->

Closes #issue_number
github-merge-queue bot pushed a commit that referenced this issue Dec 19, 2024
**Motivation**

- Support the P2P eth functionality of exchanging transaction receipts.


Closes #386 
Closes #387

---------

Co-authored-by: Martin Paulucci <martin.c.paulucci@gmail.com>
Co-authored-by: fmoletta <fedemoletta@hotmail.com>
Co-authored-by: fmoletta <99273364+fmoletta@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
network Issues related to network communication
Projects
Status: Done
3 participants