Skip to content

Commit 8212818

Browse files
fixup! Register optional dig dependency
Signed-off-by: Alexandros Filios <[email protected]>
1 parent 4ac3a6a commit 8212818

File tree

6 files changed

+45
-103
lines changed

6 files changed

+45
-103
lines changed

Diff for: platform/fabric/fns.go

+1-4
Original file line numberDiff line numberDiff line change
@@ -23,9 +23,6 @@ var (
2323
logger = logging.MustGetLogger("fabric-sdk")
2424
)
2525

26-
const DefaultNetwork = ""
27-
const DefaultChannel = ""
28-
2926
// NetworkService models a Fabric Network
3027
type NetworkService struct {
3128
subscriber events.Subscriber
@@ -189,7 +186,7 @@ func GetFabricNetworkService(sp view2.ServiceProvider, id string) (*NetworkServi
189186

190187
// GetDefaultFNS returns the default Fabric Network Service
191188
func GetDefaultFNS(sp view2.ServiceProvider) (*NetworkService, error) {
192-
return GetFabricNetworkService(sp, DefaultNetwork)
189+
return GetFabricNetworkService(sp, "")
193190
}
194191

195192
// GetDefaultChannel returns the default channel of the default fns

Diff for: platform/fabric/services/state/helpers.go

+17-12
Original file line numberDiff line numberDiff line change
@@ -21,29 +21,34 @@ func GetVaultService(ctx view2.ServiceProvider) (VaultService, error) {
2121
return s.(VaultService), nil
2222
}
2323

24-
func GetVault(ctx view2.ServiceProvider, opts ...ServiceOption) (Vault, error) {
25-
opt, err := CompileServiceOptions(opts...)
26-
if err != nil {
27-
return nil, err
28-
}
24+
func GetVault(ctx view2.ServiceProvider) (Vault, error) {
2925
vs, err := GetVaultService(ctx)
3026
if err != nil {
3127
return nil, err
3228
}
33-
if len(opt.Network) > 0 && len(opt.Channel) > 0 {
34-
return vs.Vault(opt.Network, opt.Channel)
35-
}
36-
fns, err := fabric.GetFabricNetworkService(ctx, opt.Network)
29+
fsc, ch, err := fabric.GetDefaultChannel(ctx)
3730
if err != nil {
3831
return nil, err
3932
}
40-
ch, err := fns.Channel(opt.Channel)
33+
ws, err := vs.Vault(fsc.Name(), ch.Name())
4134
if err != nil {
4235
return nil, err
4336
}
44-
return vs.Vault(fns.Name(), ch.Name())
37+
return ws, nil
4538
}
4639

4740
func GetVaultForChannel(ctx view2.ServiceProvider, channel string) (Vault, error) {
48-
return GetVault(ctx, WithChannel(channel))
41+
vs, err := GetVaultService(ctx)
42+
if err != nil {
43+
return nil, err
44+
}
45+
fns, err := fabric.GetDefaultFNS(ctx)
46+
if err != nil {
47+
return nil, err
48+
}
49+
ws, err := vs.Vault(fns.Name(), channel)
50+
if err != nil {
51+
return nil, err
52+
}
53+
return ws, nil
4954
}

Diff for: platform/fabric/services/state/recipients.go

+8-80
Original file line numberDiff line numberDiff line change
@@ -10,10 +10,8 @@ import (
1010
"encoding/json"
1111
"time"
1212

13-
"github.com/hyperledger-labs/fabric-smart-client/platform/common/utils"
1413
"github.com/hyperledger-labs/fabric-smart-client/platform/fabric"
1514
view2 "github.com/hyperledger-labs/fabric-smart-client/platform/view"
16-
"github.com/hyperledger-labs/fabric-smart-client/platform/view/driver"
1715
session2 "github.com/hyperledger-labs/fabric-smart-client/platform/view/services/session"
1816
"github.com/hyperledger-labs/fabric-smart-client/platform/view/view"
1917
"github.com/pkg/errors"
@@ -200,9 +198,6 @@ type ExchangeRecipientIdentitiesView struct {
200198
Network string
201199
IdentityLabel string
202200
Other view.Identity
203-
204-
fnsProvider *fabric.NetworkServiceProvider
205-
binder driver.EndpointService
206201
}
207202

208203
// ExchangeRecipientIdentities runs the ExchangeRecipientIdentitiesView against the passed receiver.
@@ -216,9 +211,6 @@ func ExchangeRecipientIdentities(context view.Context, recipient view.Identity,
216211
Network: opt.Network,
217212
Other: recipient,
218213
IdentityLabel: opt.Identity,
219-
220-
fnsProvider: utils.MustGet(fabric.GetNetworkServiceProvider(context)),
221-
binder: driver.GetEndpointService(context),
222214
})
223215
if err != nil {
224216
return nil, nil, err
@@ -233,7 +225,7 @@ func (f *ExchangeRecipientIdentitiesView) Call(context view.Context) (interface{
233225
return nil, err
234226
}
235227

236-
fns, err := f.fnsProvider.FabricNetworkService(f.Network)
228+
fns, err := fabric.GetFabricNetworkService(context, f.Network)
237229
if err != nil {
238230
return nil, err
239231
}
@@ -278,58 +270,25 @@ func (f *ExchangeRecipientIdentitiesView) Call(context view.Context) (interface{
278270

279271
// Update the Endpoint Resolver
280272
logger.Debugf("bind [%s] to other [%s]", recipientData.Identity, f.Other)
281-
err = f.binder.Bind(f.Other, recipientData.Identity)
273+
resolver := view2.GetEndpointService(context)
274+
err = resolver.Bind(f.Other, recipientData.Identity)
282275
if err != nil {
283276
return nil, err
284277
}
285278

286279
logger.Debugf("bind me [%s] to [%s]", me, context.Me())
287-
err = f.binder.Bind(context.Me(), me)
280+
err = resolver.Bind(context.Me(), me)
288281
if err != nil {
289282
return nil, err
290283
}
291284

292285
return []view.Identity{me, recipientData.Identity}, nil
293286
}
294287

295-
type ExchangeRecipientIdentitiesViewFactory struct {
296-
fnsProvider *fabric.NetworkServiceProvider
297-
binder driver.EndpointService
298-
}
299-
300-
func NewExchangeRecipientIdentitiesViewFactory(
301-
fnsProvider *fabric.NetworkServiceProvider,
302-
binder driver.EndpointService,
303-
) *ExchangeRecipientIdentitiesViewFactory {
304-
return &ExchangeRecipientIdentitiesViewFactory{
305-
fnsProvider: fnsProvider,
306-
binder: binder,
307-
}
308-
309-
}
310-
311-
func (f *ExchangeRecipientIdentitiesViewFactory) New(recipient view.Identity, opts ...ServiceOption) (view.View, error) {
312-
opt, err := CompileServiceOptions(opts...)
313-
if err != nil {
314-
return nil, errors.Wrapf(err, "failed to compile service options")
315-
}
316-
return &ExchangeRecipientIdentitiesView{
317-
Network: opt.Network,
318-
Other: recipient,
319-
IdentityLabel: opt.Identity,
320-
321-
fnsProvider: f.fnsProvider,
322-
binder: f.binder,
323-
}, nil
324-
}
325-
326288
// RespondExchangeRecipientIdentitiesView models the view of the responder of an exchange of recipient identities.
327289
type RespondExchangeRecipientIdentitiesView struct {
328290
Network string
329291
IdentityLabel string
330-
331-
fnsProvider *fabric.NetworkServiceProvider
332-
binder driver.EndpointService
333292
}
334293

335294
// RespondExchangeRecipientIdentities runs the RespondExchangeRecipientIdentitiesView
@@ -341,9 +300,6 @@ func RespondExchangeRecipientIdentities(context view.Context, opts ...ServiceOpt
341300
ids, err := context.RunView(&RespondExchangeRecipientIdentitiesView{
342301
Network: opt.Network,
343302
IdentityLabel: opt.Identity,
344-
345-
fnsProvider: utils.MustGet(fabric.GetNetworkServiceProvider(context)),
346-
binder: driver.GetEndpointService(context),
347303
})
348304
if err != nil {
349305
return nil, nil, err
@@ -364,7 +320,7 @@ func (s *RespondExchangeRecipientIdentitiesView) Call(context view.Context) (int
364320
return nil, err
365321
}
366322

367-
fns, err := s.fnsProvider.FabricNetworkService(s.Network)
323+
fns, err := fabric.GetFabricNetworkService(context, s.Network)
368324
if err != nil {
369325
return nil, err
370326
}
@@ -396,43 +352,15 @@ func (s *RespondExchangeRecipientIdentitiesView) Call(context view.Context) (int
396352
}
397353

398354
// Update the Endpoint Resolver
399-
err = s.binder.Bind(context.Me(), me)
355+
resolver := view2.GetEndpointService(context)
356+
err = resolver.Bind(context.Me(), me)
400357
if err != nil {
401358
return nil, err
402359
}
403-
err = s.binder.Bind(session.Info().Caller, other)
360+
err = resolver.Bind(session.Info().Caller, other)
404361
if err != nil {
405362
return nil, err
406363
}
407364

408365
return []view.Identity{me, other}, nil
409366
}
410-
411-
type RespondExchangeRecipientIdentitiesViewFactory struct {
412-
fnsProvider *fabric.NetworkServiceProvider
413-
binder driver.EndpointService
414-
}
415-
416-
func NewRespondExchangeRecipientIdentitiesViewFactory(
417-
fnsProvider *fabric.NetworkServiceProvider,
418-
binder driver.EndpointService,
419-
) *RespondExchangeRecipientIdentitiesViewFactory {
420-
return &RespondExchangeRecipientIdentitiesViewFactory{
421-
fnsProvider: fnsProvider,
422-
binder: binder,
423-
}
424-
}
425-
426-
func (f *RespondExchangeRecipientIdentitiesViewFactory) New(opts ...ServiceOption) (*RespondExchangeRecipientIdentitiesView, error) {
427-
opt, err := CompileServiceOptions(opts...)
428-
if err != nil {
429-
return nil, errors.Wrapf(err, "failed to compile service options")
430-
}
431-
return &RespondExchangeRecipientIdentitiesView{
432-
Network: opt.Network,
433-
IdentityLabel: opt.Identity,
434-
435-
fnsProvider: f.fnsProvider,
436-
binder: f.binder,
437-
}, nil
438-
}

Diff for: platform/fabric/services/state/transaction.go

+1-1
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ func NewAnonymousTransaction(context view.Context) (*Transaction, error) {
6262
// NewAnonymousTransaction returns a new instance of a state-based transaction that embeds a single namespace and is signed
6363
// by an anonymous identity
6464
func newAnonymousTransaction(fnsProvider *fabric.NetworkServiceProvider, context view.Context) (*Transaction, error) {
65-
fns, err := fnsProvider.FabricNetworkService(fabric.DefaultNetwork)
65+
fns, err := fnsProvider.FabricNetworkService("")
6666
if err != nil {
6767
return nil, err
6868
}

Diff for: platform/view/core/manager/context.go

+16-4
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,18 @@ import (
2424
"go.uber.org/zap/zapcore"
2525
)
2626

27+
type sigService interface {
28+
// IsMe returns true if a signer was ever registered for the passed identity
29+
IsMe(identity view.Identity) bool
30+
}
31+
32+
type identityProvider interface {
33+
// DefaultIdentity returns the default identity known by this provider
34+
DefaultIdentity() view.Identity
35+
// Identity returns the identity bound to the passed label
36+
Identity(label string) view.Identity
37+
}
38+
2739
type ctx struct {
2840
context context.Context
2941
sp driver.ServiceProvider
@@ -40,8 +52,8 @@ type ctx struct {
4052
sessions map[string]view.Session
4153
errorCallbackFuncs []func()
4254

43-
identityProvider driver.IdentityProvider
44-
sigService driver.SigService
55+
identityProvider identityProvider
56+
sigService sigService
4557
tracer trace.Tracer
4658
}
4759

@@ -55,7 +67,7 @@ func (ctx *ctx) StartSpanFrom(c context.Context, name string, opts ...trace.Span
5567
return ctx.tracer.Start(c, name, opts...)
5668
}
5769

58-
func NewContextForInitiator(contextID string, context context.Context, sp driver.ServiceProvider, sessionFactory SessionFactory, resolver driver.EndpointService, identityProvider driver.IdentityProvider, sigService driver.SigService, party view.Identity, initiator view.View, tracer trace.Tracer) (*ctx, error) {
70+
func NewContextForInitiator(contextID string, context context.Context, sp driver.ServiceProvider, sessionFactory SessionFactory, resolver driver.EndpointService, identityProvider identityProvider, sigService sigService, party view.Identity, initiator view.View, tracer trace.Tracer) (*ctx, error) {
5971
if len(contextID) == 0 {
6072
contextID = GenerateUUID()
6173
}
@@ -68,7 +80,7 @@ func NewContextForInitiator(contextID string, context context.Context, sp driver
6880
return ctx, nil
6981
}
7082

71-
func NewContext(context context.Context, sp driver.ServiceProvider, contextID string, sessionFactory SessionFactory, resolver driver.EndpointService, identityProvider driver.IdentityProvider, sigService driver.SigService, party view.Identity, session view.Session, caller view.Identity, tracer trace.Tracer) (*ctx, error) {
83+
func NewContext(context context.Context, sp driver.ServiceProvider, contextID string, sessionFactory SessionFactory, resolver driver.EndpointService, identityProvider identityProvider, sigService sigService, party view.Identity, session view.Session, caller view.Identity, tracer trace.Tracer) (*ctx, error) {
7284
ctx := &ctx{
7385
context: context,
7486
id: contextID,

Diff for: platform/view/core/manager/manager.go

+2-2
Original file line numberDiff line numberDiff line change
@@ -37,8 +37,8 @@ type manager struct {
3737

3838
commLayer CommLayer
3939
endpointService driver.EndpointService
40-
identityProvider driver.IdentityProvider
41-
sigService driver.SigService
40+
identityProvider identityProvider
41+
sigService sigService
4242

4343
ctx context.Context
4444

0 commit comments

Comments
 (0)