Merge pull request #114 from owncloud/init-role-service-in-grpc-server

Init role service in grpc server

changelog/unreleased/fix-role-service-grpc.md

@@ -0,0 +1,6 @@
+Bugfix: initialize roleService client in GRPC server
+
+We fixed the initialization of the GRPC server by also providing a roleService client and a roleManager instance.
+
+https://github.com/owncloud/ocis-accounts/pull/114
+

pkg/server/grpc/server.go

@@ -1,9 +1,14 @@
 package grpc
 
 import (
+	"time"
+
+	mclient "github.com/micro/go-micro/v2/client"
 	"github.com/owncloud/ocis-accounts/pkg/proto/v0"
 	svc "github.com/owncloud/ocis-accounts/pkg/service/v0"
+	"github.com/owncloud/ocis-pkg/v2/roles"
 	"github.com/owncloud/ocis-pkg/v2/service/grpc"
+	settings "github.com/owncloud/ocis-settings/pkg/proto/v0"
 )
 
 // Server initializes a new go-micro service ready to run
@@ -22,7 +27,22 @@ func Server(opts ...Option) grpc.Service {
 	var hdlr *svc.Service
 	var err error
 
-	if hdlr, err = svc.New(svc.Logger(options.Logger), svc.Config(options.Config)); err != nil {
+	// TODO this won't work with a registry other than mdns. Look into Micro's client initialization.
+	// https://github.com/owncloud/ocis-proxy/issues/38
+	rs := settings.NewRoleService("com.owncloud.api.settings", mclient.DefaultClient)
+	roleManager := roles.NewManager(
+		roles.CacheSize(1024),
+		roles.CacheTTL(time.Hour*24*7),
+		roles.Logger(options.Logger),
+		roles.RoleService(rs),
+	)
+
+	if hdlr, err = svc.New(
+		svc.Logger(options.Logger),
+		svc.Config(options.Config),
+		svc.RoleManager(&roleManager),
+		svc.RoleService(rs),
+	); err != nil {
 		options.Logger.Fatal().Err(err).Msg("could not initialize service handler")
 	}
 	if err = proto.RegisterAccountsServiceHandler(service.Server(), hdlr); err != nil {

pkg/service/v0/accounts.go

@@ -351,6 +351,9 @@ func (s Service) CreateAccount(ctx context.Context, in *proto.CreateAccountReque
 	}
 
 	// TODO: assign user role to all new users for now, as create Account request does not have any role field
+	if s.RoleService == nil {
+		return merrors.InternalServerError(s.id, "could not assign role to account: roleService not configured")
+	}
 	_, err = s.RoleService.AssignRoleToUser(ctx, &settings.AssignRoleToUserRequest{
 		AccountUuid: acc.Id,
 		RoleId:      settings_svc.BundleUUIDRoleUser,