Regression in #22216 #22227

issue hebasto opened this issue on June 11, 2021

hebasto commented at 7:28 PM on June 11, 2021: member

The recently merged #22216 causes a failed assertion:

$ src/qt/bitcoin-qt -signet
bitcoin-qt: wallet/init.cpp:127: auto WalletInit::Construct(NodeContext &)::(anonymous class)::operator()() const: Assertion `"node.args" && check' failed.
Aborted (core dumped)

hebasto added the label Bug on Jun 11, 2021
hebasto added the label Wallet on Jun 11, 2021
ryanofsky referenced this in commit d7f3b1af21 on Jun 11, 2021
ryanofsky commented at 7:41 PM on June 11, 2021: member

Sorry for this! https://github.com/bitcoin-core/gui/pull/361 should fix it
hebasto closed this on Jun 11, 2021
hebasto referenced this in commit a8c8dbc98f on Jun 11, 2021
sidhujag referenced this in commit f194dc2845 on Jun 12, 2021
rebroad referenced this in commit 352d3fcc13 on Jun 23, 2021
rebroad referenced this in commit 7cea05d4e7 on Jun 28, 2021
janus referenced this in commit 64af71c62b on Oct 29, 2021
DrahtBot locked this on Aug 18, 2022

Contributors

hebasto

ryanofsky

Labels

Bug Wallet

Linked (view graph)

#22216 refactor: Make SetupServerArgs callable without NodeContext