gaspersic/solana
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Fix signature subscription panic (#12077) (#12092)

Browse Source
This commit is contained in:
Justin Starry
2020-09-07 18:31:25 +08:00
committed by GitHub
parent f3f86f43ee
commit 58312655b4
1 changed files with 2 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
core/src/rpc_subscriptions.rs
View File

@ -634,7 +634,7 @@ impl RpcSubscriptions {
) { ) {
let (commitment, enable_received_notification) = signature_subscribe_config let (commitment, enable_received_notification) = signature_subscribe_config
.map(|config| (config.commitment, config.enable_received_notification)) .map(|config| (config.commitment, config.enable_received_notification))
.unwrap_or((None, Some(false))); .unwrap_or_default();
let commitment_level = commitment let commitment_level = commitment
.unwrap_or_else(CommitmentConfig::recent) .unwrap_or_else(CommitmentConfig::recent)
@ -970,9 +970,7 @@ impl RpcSubscriptions {
}, },
) in hashmap.iter() ) in hashmap.iter()
{ {
if is_received_notification_enabled if is_received_notification_enabled.unwrap_or_default() {
.expect("All signature subscriptions must have this config field set")
{
notifier.notify( notifier.notify(
Response { Response {
context: RpcResponseContext { context: RpcResponseContext {