Cannot use `-rpcuser` and `-rpcpassword` as arguments, does not return Err and enters infinite loop about bitcoind HOT 6 CLOSED

I need to use a custom user and password but didn't notice that it's deprecated, using the -rpcauth works fine.

Great, a specific error would've saved me some time 😅

I'll work in a PR candidate to fix and update to give a proper error.

from bitcoind.

@RCasatta Please let me know if the bug makes sense or was expected behavior, and if any more info is needed.

If it either needs to fix to return an error or to accept the arguments and not consider the .cookie file I will be more than happy to discuss a solution, build the fix, and open a PR.

from bitcoind.

cookie support dates back to bitcoin core version 0.12, do you need to test with versions prior to this?

I think we should probably prevent users to specify rpcuser and rpcpassword and give a proper error.

from bitcoind.

Thanks for opening this issue - I was using rpcuser / rpcpassword, and seeing that there were problems with it saved me a bunch of debugging. If rpcuser / rpcpassword are known not to work, perhaps it should be documented somewhere? I had the assumption that although those options were deprecated in bitcoind, they would still be generally work, as this crate seems to be a relatively thin wrapper around bitcoind.

from bitcoind.

@MaxFangX I'm really glad that the issue helped and saved you a lot of time, I've used a lot of debugging time to figure out the problem 😅

I've not finished my work on a draft PR due to my final exams and SoB project

I'll try to open it in the next few days to add a better error message as discussed with RCasatta above.

from bitcoind.

@RCasatta I've now opened PR #72 related to this issue, would really appreciate a review.

I think would also be useful to have a function to produce the -rpcauth expected value, wdyt ?

I can port the bitcoin core python script like this one: https://github.com/testcontainers/testcontainers-rs/blob/dev/testcontainers/src/images/coblox_bitcoincore.rs#L40-L91

cc @MaxFangX

from bitcoind.