Merge pull request #3573 from wpaulino/log-conf-ntfn-once

chainntnfs: only log confirmation notification registration once
This commit is contained in:
Johan T. Halseth 2019-10-04 13:06:42 +02:00 committed by GitHub
commit e88c635b7e
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

@ -601,9 +601,6 @@ func (n *TxNotifier) RegisterConf(txid *chainhash.Hash, pkScript []byte,
return nil, err
}
Log.Infof("New confirmation subscription: %v, num_confs=%v ",
ntfn.ConfRequest, numConfs)
// Before proceeding to register the notification, we'll query our
// height hint cache to determine whether a better one exists.
//
@ -622,12 +619,13 @@ func (n *TxNotifier) RegisterConf(txid *chainhash.Hash, pkScript []byte,
ntfn.ConfRequest, err)
}
Log.Infof("New confirmation subscription: conf_id=%d, %v, "+
"num_confs=%v height_hint=%d", ntfn.ConfID, ntfn.ConfRequest,
numConfs, startHeight)
n.Lock()
defer n.Unlock()
Log.Infof("New confirmation subscription: conf_id=%d, %v, "+
"height_hint=%d", ntfn.ConfID, ntfn.ConfRequest, startHeight)
confSet, ok := n.confNotifications[ntfn.ConfRequest]
if !ok {
// If this is the first registration for this request, construct