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

sequenced name_updates fails #309

Open
RyuMaster opened this issue Mar 28, 2022 · 3 comments
Open

sequenced name_updates fails #309

RyuMaster opened this issue Mar 28, 2022 · 3 comments

Comments

@RyuMaster
Copy link

If I sent one name_update after another, electrum returns 500 error for the second one, only first name_update is being executed
Both only works, if I have decent-enough delay between, like 3 seconds or more.

Is it something I need to configure additionally?

@JeremyRand
Copy link
Member

Are you using the broadcast command by itself to send the transactions, or are you also using addtransaction? There is an undocumented race condition in upstream Electrum; the former will only result in your wallet learning about the transaction after the server reports it, which would explain your 3-second minimum. If that's the cause, let me know and I can look into documenting this upstream.

@domob1812
Copy link

We are using broadcast, so that explains it indeed. Thanks for pointing this out! If we change that to addtransaction, will that automatically broadcast, or do we need to do both of them?

@JeremyRand
Copy link
Member

You need to do both; see the implementation of the name_autoregister command for an example. Thanks for confirming this is the cause; I'll check with upstream about documenting this better. (I'll leave this issue open until the documentation is fixed.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants