-
Notifications
You must be signed in to change notification settings - Fork 38.8k
Bugfix: RPC/mining: Fail properly in estimatesmartfee if smart fee data is unavailable #23547
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
Bugfix: RPC/mining: Fail properly in estimatesmartfee if smart fee data is unavailable #23547
Conversation
…ta is unavailable
ghost
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
crACK cd8d156
|
maybe worth for 22.1 |
darosior
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Arf, good catch..
utACK cd8d156
kristapsk
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
utACK cd8d156
|
A regression test might be nice |
…rtfee if smart fee data is unavailable cd8d156 Bugfix: RPC/mining: Fail properly in estimatesmartfee if smart fee data is unavailable (Luke Dashjr) Pull request description: Fixes a regression introduced by bitcoin#22722 (Not entirely sure on the solution) ACKs for top commit: prayank23: crACK bitcoin@cd8d156 darosior: utACK cd8d156 kristapsk: utACK cd8d156 Tree-SHA512: eb4aa3cc345c69c44ffd5733b51b90eefe1d7854b7a2855e8cbb98268db24d43b7d0ae9fbb0eccf9b6dc01da644d19433cc77fec52ff67bf890be1fc53a67fc4
…ta is unavailable Github-Pull: bitcoin#23547 Rebased-From: cd8d156
…imatesmartfee if smart fee data is unavailable 9ed340f Bugfix: RPC/mining: Fail properly in estimatesmartfee if smart fee data is unavailable (Luke Dashjr) Pull request description: Fixes a regression introduced by #22722 (Not entirely sure on the solution) ACKs for top commit: prayank23: crACK bitcoin/bitcoin@9ed340f darosior: utACK 9ed340f kristapsk: utACK 9ed340f Tree-SHA512: eb4aa3cc345c69c44ffd5733b51b90eefe1d7854b7a2855e8cbb98268db24d43b7d0ae9fbb0eccf9b6dc01da644d19433cc77fec52ff67bf890be1fc53a67fc4
Fixes a regression introduced by #22722
(Not entirely sure on the solution)