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
ProcessBridge refactoring & bugfix #3858
ProcessBridge refactoring & bugfix #3858
Conversation
063e9e2
to
adbbd9a
Compare
using var cts = new CancellationTokenSource(); | ||
cts.Cancel(); | ||
var processBridge = new ProcessBridge(MicroserviceHelpers.GetBinaryPath("bitcoind")); | ||
(string response, int exitCode) p = await processBridge.SendCommandAsync("-testnet", openConsole: false, cts.Token); |
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.
Is this supposed to be --testnet
or -testnet
?
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.
For bitcoind the switch is -testnet
4ce73c8
to
e1e161e
Compare
56867d1
to
b5977d4
Compare
b5977d4
to
2ada212
Compare
ba6fb25
to
528e007
Compare
054221c
to
200d1fc
Compare
Co-authored-by: yahiheb <52379387+yahiheb@users.noreply.github.com>
Co-authored-by: yahiheb <52379387+yahiheb@users.noreply.github.com>
200d1fc
to
92e3a4f
Compare
@molnard @lontivero Wdyt? It contains a bugfix but it's also large change. I would say, it's better to wait with this after release given that the bug should manifest itself very rarely. Finished refactoring is here: #3863 but it's extra large. |
Yes, it looks like a big change and IMO the discussion on this will be too. After the release... |
Superseded by #3863. |
This PR is supposed to fix: #3852 (comment)
It should be probably discussed after the Wasabi release.
Follow-up:
ProcessBridge
. Without it, it's necessary to add new arguments whenever any new functionality is needed (example)