Skip to content

Commit be54c41

Browse files
committed
Forward requests from stubs to pkg
1 parent f831376 commit be54c41

File tree

1 file changed

+34
-9
lines changed

1 file changed

+34
-9
lines changed

internal/grpc/services/appregistry/appregistry.go

+34-9
Original file line numberDiff line numberDiff line change
@@ -115,9 +115,17 @@ func (s *svc) GetAppProviders(ctx context.Context, req *registrypb.GetAppProvide
115115
}
116116

117117
func (s *svc) AddAppProvider(ctx context.Context, req *registrypb.AddAppProviderRequest) (*registrypb.AddAppProviderResponse, error) {
118-
return &registrypb.AddAppProviderResponse{
119-
Status: status.NewUnimplemented(ctx, errtypes.NotSupported("AddAppProvider not implemented"), "AddAppProvider not implemented"),
120-
}, nil
118+
err := s.reg.AddProvider(ctx, req.Provider)
119+
if err != nil {
120+
return &registrypb.AddAppProviderResponse{
121+
Status: status.NewInternal(ctx, err, "error adding the app provider"),
122+
}, nil
123+
}
124+
125+
res := &registrypb.AddAppProviderResponse{
126+
Status: status.NewOK(ctx),
127+
}
128+
return res, nil
121129
}
122130

123131
func (s *svc) ListAppProviders(ctx context.Context, req *registrypb.ListAppProvidersRequest) (*registrypb.ListAppProvidersResponse, error) {
@@ -136,13 +144,30 @@ func (s *svc) ListAppProviders(ctx context.Context, req *registrypb.ListAppProvi
136144
}
137145

138146
func (s *svc) GetDefaultAppProviderForMimeType(ctx context.Context, req *registrypb.GetDefaultAppProviderForMimeTypeRequest) (*registrypb.GetDefaultAppProviderForMimeTypeResponse, error) {
139-
return &registrypb.GetDefaultAppProviderForMimeTypeResponse{
140-
Status: status.NewUnimplemented(ctx, errtypes.NotSupported("GetDefaultAppProviderForMimeType not implemented"), "GetDefaultAppProviderForMimeType not implemented"),
141-
}, nil
147+
provider, err := s.reg.GetDefaultProviderForMimeType(ctx, req.MimeType)
148+
if err != nil {
149+
return &registrypb.GetDefaultAppProviderForMimeTypeResponse{
150+
Status: status.NewInternal(ctx, err, "error getting the default app provider for the mimetype"),
151+
}, nil
152+
}
153+
154+
res := &registrypb.GetDefaultAppProviderForMimeTypeResponse{
155+
Status: status.NewOK(ctx),
156+
Provider: provider,
157+
}
158+
return res, nil
142159
}
143160

144161
func (s *svc) SetDefaultAppProviderForMimeType(ctx context.Context, req *registrypb.SetDefaultAppProviderForMimeTypeRequest) (*registrypb.SetDefaultAppProviderForMimeTypeResponse, error) {
145-
return &registrypb.SetDefaultAppProviderForMimeTypeResponse{
146-
Status: status.NewUnimplemented(ctx, errtypes.NotSupported("SetDefaultAppProviderForMimeType not implemented"), "SetDefaultAppProviderForMimeType not implemented"),
147-
}, nil
162+
err := s.reg.SetDefaultProviderForMimeType(ctx, req.MimeType, req.Provider)
163+
if err != nil {
164+
return &registrypb.SetDefaultAppProviderForMimeTypeResponse{
165+
Status: status.NewInternal(ctx, err, "error setting the default app provider for the mimetype"),
166+
}, nil
167+
}
168+
169+
res := &registrypb.SetDefaultAppProviderForMimeTypeResponse{
170+
Status: status.NewOK(ctx),
171+
}
172+
return res, nil
148173
}

0 commit comments

Comments
 (0)