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.

Sign up for GitHub

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

Jump to bottom

Send inbound MempoolTransactionIds requests to the mempool storage service #1083

Closed
Tracked by #2309
hdevalence opened this issue Sep 20, 2020 · 0 comments · Fixed by #2720
Closed
Tracked by #2309

Send inbound MempoolTransactionIds requests to the mempool storage service #1083

hdevalence opened this issue Sep 20, 2020 · 0 comments · Fixed by #2720
Assignees
@oxarbitrage
Labels
A-rust Area: Updates to Rust code C-enhancement Category: This is an improvement
Milestone
2021 Sprint 17

Comments

@hdevalence
Copy link
Contributor

hdevalence commented Sep 20, 2020 •
edited by teor2345
Loading

Is your feature request related to a problem? Please describe.

The inbound network service should handle MempoolTransactionIds requests correctly.

Describe the solution you'd like

Query mempool storage for accepted transactions, and return those UnminedTxIds.

Additional context

Part of #889.

https://developer.bitcoin.org/reference/p2p_networking.html#mempool
@hdevalence hdevalence added E-easy A-rust Area: Updates to Rust code S-blocked Status: Blocked on other tasks C-enhancement Category: This is an improvement labels Sep 20, 2020
@hdevalence hdevalence mentioned this issue Sep 20, 2020
Closed
11 tasks
@teor2345 teor2345 added this to the Sync and Network milestone Sep 29, 2020
@mpguerra mpguerra removed this from the Sync and Network milestone Jan 5, 2021
@mpguerra mpguerra removed the E-easy label Mar 23, 2021
@mpguerra mpguerra added this to the 2021 Sprint 17 milestone Aug 9, 2021
@mpguerra mpguerra mentioned this issue Aug 11, 2021
Closed
59 tasks
@teor2345 teor2345 changed the title Handle MempoolTransactions in the Inbound service Send inbound MempoolTransactionIds to the mempool storage service Aug 17, 2021
@teor2345 teor2345 removed the S-blocked Status: Blocked on other tasks label Aug 17, 2021
@teor2345 teor2345 changed the title Send inbound MempoolTransactionIds to the mempool storage service Send inbound mempool TransactionIds to the mempool storage service Aug 17, 2021
@teor2345 teor2345 changed the title Send inbound mempool TransactionIds to the mempool storage service Send inbound MempoolTransactionIds requests to the mempool storage service Aug 17, 2021
@oxarbitrage oxarbitrage self-assigned this Aug 23, 2021
@oxarbitrage oxarbitrage mentioned this issue Sep 1, 2021
Merged
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@oxarbitrage oxarbitrage

Labels
A-rust Area: Updates to Rust code C-enhancement Category: This is an improvement
Projects
None yet
Milestone
2021 Sprint 17
Development

Successfully merging a pull request may close this issue.

Reply to Request::MempoolTransactionIds with mempool content oxarbitrage/zebra
4 participants
@hdevalence @mpguerra @teor2345 @oxarbitrage