Skip to content

Commit d285717

Browse files
committed
cleanup
Signed-off-by: Angelo De Caro <adc@zurich.ibm.com>
1 parent ce4428b commit d285717

4 files changed

Lines changed: 32 additions & 28 deletions

File tree

integration/token/fungible/dlogx/dlog_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -86,7 +86,7 @@ func newTestSuite(commType fsc.P2PCommunicationType, mask int, factor int, token
8686
Monitoring: false,
8787
ReplicationOpts: opts,
8888
FSCBasedEndorsement: mask&WithEndorsers > 0,
89-
FSCLogSpec: "fts=debug:grpc=error:info",
89+
FSCLogSpec: "grpc=error:info",
9090
TokenSelector: tokenSelector,
9191
})...)
9292
i.RegisterPlatformFactory(fabricx.NewPlatformFactory())

integration/token/fungible/support.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1046,7 +1046,7 @@ func UpdatePublicParamsAndWait(network *integration.Infrastructure, publicParams
10461046
if node.Id() == "custodian" {
10471047
continue
10481048
}
1049-
gomega.Eventually(GetPublicParams).WithArguments(network, node).WithTimeout(30 * time.Second).WithPolling(15 * time.Second).Should(gomega.Equal(publicParams))
1049+
gomega.Eventually(GetPublicParams).WithArguments(network, node).WithTimeout(60 * time.Second).WithPolling(5 * time.Second).Should(gomega.Equal(publicParams))
10501050
}
10511051
}
10521052

token/services/network/fabricx/lookup/gocron.go

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,12 +9,14 @@ package lookup
99
import (
1010
"bytes"
1111
"context"
12+
"encoding/base64"
1213
"sync"
1314
"time"
1415

1516
"github.com/go-co-op/gocron/v2"
1617
"github.com/hyperledger-labs/fabric-smart-client/pkg/utils/errors"
1718
"github.com/hyperledger-labs/fabric-smart-client/platform/fabricx/core/vault/queryservice"
19+
"github.com/hyperledger-labs/fabric-token-sdk/token"
1820
"github.com/hyperledger-labs/fabric-token-sdk/token/services/logging"
1921
"github.com/hyperledger-labs/fabric-token-sdk/token/services/network/fabric/lookup"
2022
)
@@ -205,10 +207,11 @@ func (j *PermanentJob) Run() {
205207
logger.Infof("[PermanentKeyCheck] check for key [%s:%s]", j.namespace, j.key)
206208
v, err := j.queryService.GetState(j.namespace, j.key)
207209
if err == nil && v != nil && len(v.Raw) != 0 {
208-
if !bytes.Equal(j.lastValue, v.Raw) {
209-
logger.Infof("[PermanentKeyCheck] key [%s:%s] found with new value, notify listener", j.namespace, j.key)
210+
newHash := token.Hashable(v.Raw).Raw()
211+
if !bytes.Equal(j.lastValue, newHash) {
212+
logger.Debugf("[PermanentKeyCheck] key [%s:%s] found with new value [%s], notify listener", j.namespace, j.key, base64.StdEncoding.EncodeToString(newHash))
210213
j.listener.OnStatus(context.Background(), j.key, v.Raw)
211-
j.lastValue = v.Raw
214+
j.lastValue = token.Hashable(v.Raw).Raw()
212215
}
213216
}
214217
}

token/services/network/fabricx/pp/service.go

Lines changed: 24 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -7,25 +7,30 @@ SPDX-License-Identifier: Apache-2.0
77
package pp
88

99
import (
10-
"fmt"
11-
"reflect"
12-
10+
"github.com/hyperledger-labs/fabric-smart-client/pkg/utils/errors"
1311
"github.com/hyperledger-labs/fabric-smart-client/platform/common/driver"
14-
"github.com/hyperledger-labs/fabric-smart-client/platform/view/services"
12+
"github.com/hyperledger-labs/fabric-smart-client/platform/fabricx/core/vault/queryservice"
1513
"github.com/hyperledger-labs/fabric-token-sdk/token"
1614
"github.com/hyperledger-labs/fabric-token-sdk/token/services/logging"
15+
"github.com/hyperledger-labs/fabric-token-sdk/token/services/network/common/rws/keys"
1716
)
1817

1918
var logger = logging.MustGetLogger()
2019

2120
// PublicParametersService models a service for fetching and loading public parameters.
2221
type PublicParametersService struct {
2322
tmsProvider *token.ManagementServiceProvider
23+
qsProvider queryservice.Provider
24+
translator *keys.Translator
2425
}
2526

2627
// NewPublicParametersService returns a new PublicParametersService instance.
27-
func NewPublicParametersService(tmsProvider *token.ManagementServiceProvider) *PublicParametersService {
28-
return &PublicParametersService{tmsProvider: tmsProvider}
28+
func NewPublicParametersService(tmsProvider *token.ManagementServiceProvider, qsProvider queryservice.Provider) *PublicParametersService {
29+
return &PublicParametersService{
30+
tmsProvider: tmsProvider,
31+
qsProvider: qsProvider,
32+
translator: &keys.Translator{},
33+
}
2934
}
3035

3136
// LoadPublicParams loads the public parameters for the given TMS ID.
@@ -35,31 +40,27 @@ func (f *PublicParametersService) LoadPublicParams(tmsID token.TMSID, ppRaw []by
3540

3641
// Fetch returns the public parameters for the given network, channel, and namespace.
3742
func (f *PublicParametersService) Fetch(network driver.Network, channel driver.Channel, namespace driver.Namespace) ([]byte, error) {
38-
tmsID := token.TMSID{
39-
Network: network,
40-
Channel: channel,
41-
Namespace: namespace,
43+
qs, err := f.qsProvider.Get(network, channel)
44+
if err != nil {
45+
return nil, errors.Wrapf(err, "failed getting query service")
4246
}
43-
logger.Infof("Fetch public params for [%s]", tmsID)
44-
tms, err := f.tmsProvider.GetManagementService(token.WithTMSID(tmsID))
47+
k, err := f.translator.CreateSetupKey()
4548
if err != nil {
46-
return nil, fmt.Errorf("tms [%s] not found: %w", tmsID, err)
49+
return nil, errors.Wrapf(err, "failed creating setup key")
50+
}
51+
value, err := qs.GetState(namespace, k)
52+
if err != nil {
53+
return nil, errors.Wrapf(err, "failed getting state")
54+
}
55+
if value == nil {
56+
return nil, nil
4757
}
4858

49-
return tms.PublicParametersManager().PublicParameters().Serialize()
59+
return value.Raw, nil
5060
}
5161

5262
// Loader models a loader for public parameters.
5363
type Loader interface {
5464
// LoadPublicParams loads the public parameters for the given TMS ID.
5565
LoadPublicParams(TMSID token.TMSID, ppRaw []byte) error
5666
}
57-
58-
func GetPublicParametersService(sp services.Provider) (Loader, error) {
59-
s, err := sp.GetService(reflect.TypeOf((*Loader)(nil)))
60-
if err != nil {
61-
return nil, err
62-
}
63-
64-
return s.(Loader), nil
65-
}

0 commit comments

Comments
 (0)