Skip to content

Commit f6c7898

Browse files
committed
AddFinalityListener remove namespace arg
Signed-off-by: Angelo De Caro <adc@zurich.ibm.com>
1 parent c167f63 commit f6c7898

File tree

8 files changed

+18
-28
lines changed

8 files changed

+18
-28
lines changed

integration/token/common/views/finality.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ func (r *TxFinalityView) Call(context view.Context) (interface{}, error) {
4444
assert.NoError(err)
4545
nw := network.GetInstance(context, tms.Network(), tms.Channel())
4646
assert.NotNil(nw)
47-
assert.NoError(nw.AddFinalityListener(tms.Namespace(), r.TxID, newFinalityListener(r.Timeout, errs)))
47+
assert.NoError(nw.AddFinalityListener(r.TxID, newFinalityListener(r.Timeout, errs)))
4848

4949
// Listen for finality from DBs
5050
go func() {

token/services/auditor/auditor.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -126,7 +126,7 @@ func (a *Service) Append(ctx context.Context, tx Transaction) error {
126126
a.tokenDB,
127127
a.finalityTracer,
128128
)
129-
if err := net.AddFinalityListener(tx.Namespace(), tx.ID(), r); err != nil {
129+
if err := net.AddFinalityListener(tx.ID(), r); err != nil {
130130
return errors.WithMessagef(err, "failed listening to network [%s:%s]", tx.Network(), tx.Channel())
131131
}
132132
logger.DebugfContext(ctx, "append done for request [%s]", tx.ID())

token/services/auditor/manager.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -120,7 +120,7 @@ func (cm *ServiceManager) RestoreTMS(tmsID token.TMSID) error {
120120

121121
return iterators.ForEach(it, func(record *storage.TokenRequestRecord) error {
122122
logger.Debugf("restore transaction [%s] with status [%s]", record.TxID, TxStatusMessage[record.Status])
123-
return net.AddFinalityListener(tmsID.Namespace, record.TxID, finality.NewListener(logger, cm.tmsProvider, tmsID, auditor.auditDB, tokenDB, auditor.finalityTracer))
123+
return net.AddFinalityListener(record.TxID, finality.NewListener(logger, cm.tmsProvider, tmsID, auditor.auditDB, tokenDB, auditor.finalityTracer))
124124
})
125125
}
126126

token/services/network/network.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -249,8 +249,8 @@ func (n *Network) LocalMembership() *LocalMembership {
249249
// When the listener is invoked, then it is also removed.
250250
// If the transaction id is empty, the listener will be called on status changes of any transaction.
251251
// In this case, the listener is not removed
252-
func (n *Network) AddFinalityListener(namespace string, txID string, listener FinalityListener) error {
253-
return n.n.AddFinalityListener(namespace, txID, listener)
252+
func (n *Network) AddFinalityListener(txID string, listener FinalityListener) error {
253+
return n.n.AddFinalityListener("", txID, listener)
254254
}
255255

256256
// RemoveFinalityListener unregisters the passed listener.

token/services/ttx/db.go

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -62,11 +62,7 @@ func (a *Service) Append(ctx context.Context, tx *Transaction) error {
6262
}
6363
logger.DebugfContext(ctx, "register tx status listener for tx [%s:%s] at network", tx.ID(), tx.Network())
6464

65-
if err := net.AddFinalityListener(
66-
tx.Namespace(),
67-
tx.ID(),
68-
finality.NewListener(logger, a.tmsProvider, a.tmsID, a.ttxStoreService, a.tokensService, a.finalityTracer),
69-
); err != nil {
65+
if err := net.AddFinalityListener(tx.ID(), finality.NewListener(logger, a.tmsProvider, a.tmsID, a.ttxStoreService, a.tokensService, a.finalityTracer)); err != nil {
7066
return errors.WithMessagef(err, "failed listening to network [%s:%s]", tx.Network(), tx.Channel())
7167
}
7268
logger.DebugfContext(ctx, "append done for request %s", tx.ID())

token/services/ttx/dep/mock/network.go

Lines changed: 10 additions & 12 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

token/services/ttx/dep/providers.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ var (
2929
//
3030
//go:generate counterfeiter -o mock/network.go -fake-name Network . Network
3131
type Network interface {
32-
AddFinalityListener(namespace string, txID string, listener network.FinalityListener) error
32+
AddFinalityListener(txID string, listener network.FinalityListener) error
3333
NewEnvelope() *network.Envelope
3434
AnonymousIdentity() (view.Identity, error)
3535
LocalMembership() *network.LocalMembership

token/services/ttx/manager.go

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -108,11 +108,7 @@ func (m *ServiceManager) RestoreTMS(ctx context.Context, tmsID token.TMSID) erro
108108
}
109109
return iterators.ForEach(it, func(record *storage.TokenRequestRecord) error {
110110
logger.Debugf("restore transaction [%s] with status [%s]", record.TxID, TxStatusMessage[record.Status])
111-
return net.AddFinalityListener(
112-
tmsID.Namespace,
113-
record.TxID,
114-
finality.NewListener(logger, db.tmsProvider, db.tmsID, db.ttxStoreService, db.tokensService, db.finalityTracer),
115-
)
111+
return net.AddFinalityListener(record.TxID, finality.NewListener(logger, db.tmsProvider, db.tmsID, db.ttxStoreService, db.tokensService, db.finalityTracer))
116112
})
117113
}
118114

0 commit comments

Comments
 (0)