Client: Drop script subscriptions to free lock

TheCharlatan · TheCharlatan · commit aa0b9578188d · 2022-08-16T22:21:47.000+02:00
This fixes a supposed deadlock when the reader thread receives a script
subscription notification from the server before it receives a
subscribing response
diff --git a/src/raw_client.rs b/src/raw_client.rs
@@ -819,6 +819,7 @@ impl<T: Read + Write> ElectrumApi for RawClient<T> {
         }
 
         script_notifications.insert(script_hash, VecDeque::new());
+        drop(script_notifications);
 
         let req = Request::new_id(
             self.last_id.fetch_add(1, Ordering::SeqCst),

Original file line number	Diff line number	Diff line change
`@@ -819,6 +819,7 @@ impl<T: Read + Write> ElectrumApi for RawClient<T> {`
`819`	`819`	`}`
`820`	`820`
`821`	`821`	`script_notifications.insert(script_hash, VecDeque::new());`
	`822`	`+ drop(script_notifications);`
`822`	`823`
`823`	`824`	`let req = Request::new_id(`
`824`	`825`	`self.last_id.fetch_add(1, Ordering::SeqCst),`