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

Don't return MWEB-only transactions when rpcserialversion <= 1 #866

Open
DavidBurkett opened this issue Oct 13, 2022 · 2 comments · May be fixed by #885
Open

Don't return MWEB-only transactions when rpcserialversion <= 1 #866

DavidBurkett opened this issue Oct 13, 2022 · 2 comments · May be fixed by #885
Assignees
@DavidBurkett
Labels
MWEB

Comments

@DavidBurkett
Copy link
Member

spesmilo/electrumx#179 (comment)
@DavidBurkett DavidBurkett self-assigned this Oct 13, 2022
@litecoin-project litecoin-project deleted a comment Nov 20, 2022
@ynohtna92
Copy link

Any chance we can get this as a priority for the next release?

@DavidBurkett DavidBurkett linked a pull request Mar 13, 2023 that will close this issue
Open
@DavidBurkett
Copy link
Member Author

Any chance we can get this as a priority for the next release?

@ynohtna92 Yes! Can you take a look at this PR and see if it resolves the issue for you? We're still returning tx id's for the mempool entries from getrawmempool, but we return an RPC error when trying to query for MWEB-only transactions using getmempoolentry. From what I can tell after a quick glance at the electrumx code, that should work for you.
#885

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@DavidBurkett DavidBurkett

Labels
MWEB
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Don't return MWEB-only transactions when rpcserialversion <= 1 litecoin-project/litecoin
2 participants
@ynohtna92 @DavidBurkett