Merge #273: Derive Deserialize on ImportDescriptors

df2ab99 Remove lifetime (Casey Rodarmor)
edfd3eb Derive Deserialize on ImportDescriptors (Casey Rodarmor)

Pull request description:

  We wound up needing `Deserialize` on `ImportDescriptors` for our mock Bitcoin Core RPC server.

ACKs for top commit:
  apoelstra:
    ACK df2ab99

Tree-SHA512: 560e4eacad8c410c80cb28a89c8c0346be3864a97545ac2d274a76d21472dc045b03f1fdb328a657757b923c8a7fec512699a2f354a6f8f8019efd71abd6fb18

Author: apoelstra

json/src/lib.rs

@@ -1254,10 +1254,10 @@ pub struct ImportMultiResult {
 }
 
 /// A import request for importdescriptors.
-#[derive(Clone, PartialEq, Eq, Debug, Default, Serialize)]
-pub struct ImportDescriptors<'a> {
+#[derive(Clone, PartialEq, Eq, Debug, Default, Deserialize, Serialize)]
+pub struct ImportDescriptors {
     #[serde(rename = "desc")]
-    pub descriptor: &'a str,
+    pub descriptor: String,
     pub timestamp: Timestamp,
     #[serde(skip_serializing_if = "Option::is_none")]
     pub active: Option<bool>,
@@ -1268,7 +1268,7 @@ pub struct ImportDescriptors<'a> {
     #[serde(skip_serializing_if = "Option::is_none")]
     pub internal: Option<bool>,
     #[serde(skip_serializing_if = "Option::is_none")]
-    pub label: Option<&'a str>,
+    pub label: Option<String>,
 }
 
 /// Progress toward rejecting pre-softfork blocks