Merge pull request #117 from renproject/update-bitcoin-client

Fix: Update bitcoin client for race condition fixed in later release

Author: roynalnaruto

infra/bitcoin/Dockerfile

@@ -6,8 +6,8 @@ RUN apt-get install --yes curl
 # bitcoind PPA is unmaintained, so we download it from the bitcoincore website.
 # This can sometimes stall, which can be fixed by killing and restarting the
 # build.
-RUN curl https://bitcoincore.org/bin/bitcoin-core-0.19.1/bitcoin-0.19.1-x86_64-linux-gnu.tar.gz | tar -xz
-RUN mv ./bitcoin-0.19.1 /app
+RUN curl https://bitcoincore.org/bin/bitcoin-core-0.21.0/bitcoin-0.21.0-x86_64-linux-gnu.tar.gz | tar -xz
+RUN mv ./bitcoin-0.21.0 /app
 RUN chmod +x /app/bin/bitcoind
 RUN chmod +x /app/bin/bitcoin-cli
 

infra/bitcoin/run.sh

@@ -3,28 +3,31 @@ ADDRESS=$1
 PRIV_KEY=$2
 
 # Start
-/app/bin/bitcoind
-sleep 10
+/app/bin/bitcoind -regtest -daemon
+sleep 20
 
 # Print setup
 echo "BITCOIN_ADDRESS=$ADDRESS"
 
+# Create wallet
+/app/bin/bitcoin-cli createwallet "testwallet"
+
 # Import the address
-/app/bin/bitcoin-cli importaddress $ADDRESS
+/app/bin/bitcoin-cli -regtest importaddress $ADDRESS
 
 # Import the private key to spend UTXOs
-/app/bin/bitcoin-cli importprivkey $PRIV_KEY
+/app/bin/bitcoin-cli -regtest importprivkey $PRIV_KEY
 
 # Generate enough block to pass the maturation time
-/app/bin/bitcoin-cli generatetoaddress 101 $ADDRESS
+/app/bin/bitcoin-cli -regtest generatetoaddress 101 $ADDRESS
 
 # Simulate mining
 while :
 do
     # generate new btc to the address
-    /app/bin/bitcoin-cli generatetoaddress 1 $ADDRESS
+    /app/bin/bitcoin-cli -regtest generatetoaddress 1 $ADDRESS
     sleep 5
     # send tx to own address while paying fee to the miner
-    /app/bin/bitcoin-cli sendtoaddress $ADDRESS 0.5 "" "" true
+    /app/bin/bitcoin-cli -regtest sendtoaddress $ADDRESS 1 "" "" true
     sleep 5
 done