Add derivation_index to GetAddressResult

The derivation index is required for
for client to derive and verify the address
on hardware wallets.

src/commands/mod.rs

@@ -296,7 +296,7 @@ impl DaemonControl {
             .receive_descriptor()
             .derive(index, &self.secp)
             .address(self.config.bitcoin_config.network);
-        GetAddressResult::new(address)
+        GetAddressResult::new(address, index.into())
     }
 
     /// list addresses
@@ -915,6 +915,7 @@ pub struct GetInfoResult {
 pub struct GetAddressResult {
     #[serde(deserialize_with = "deser_addr_assume_checked")]
     address: bitcoin::Address,
+    derivation_index: u32,
 }
 
 #[derive(Debug, Clone, Serialize, Deserialize)]
@@ -923,8 +924,11 @@ pub struct GetLabelsResult {
 }
 
 impl GetAddressResult {
-    pub fn new(address: bitcoin::Address) -> Self {
-        Self { address }
+    pub fn new(address: bitcoin::Address, derivation_index: u32) -> Self {
+        Self {
+            address,
+            derivation_index,
+        }
     }
 
     pub fn address(&self) -> &bitcoin::Address {

tests/test_rpc.py

@@ -35,6 +35,7 @@ def test_getaddress(lianad):
     assert "address" in res
     # We'll get a new one at every call
     assert res["address"] != lianad.rpc.getnewaddress()["address"]
+    assert res["derivation_index"] > lianad.rpc.getnewaddress()["derivation_index"]
 
 
 def test_listaddresses(lianad):