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

Fix signaure subscription panic (#12077)

Browse Source
This commit is contained in:
Justin Starry
2020-09-06 14:33:12 +08:00
committed by GitHub
parent 3b1dbfcce2
commit 369d376d10
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 {