Refactor controller grpc api for better separation of concern

See merge request !298

controller/controller.go

@@ -103,12 +103,12 @@ func initialize() error {
 }
 
 func startGrpc() error {
-	sock := viper.GetString("socket")
-	lis, err := net.Listen("tcp", sock)
+	socket := viper.GetString("socket")
+	lislisten, err := net.Listen("tcp", socket)
 	if err != nil {
 		return err
 	}
-	log.Infof("listening to %v", lis.Addr())
+	log.Infof("listening to %v", lislisten.Addr())
 
 	jwtManager := rbacImpl.NewJWTManager("", (10000 * time.Hour)) //TODO(faseid): add real secret and proper duration data here!
 	setupGRPCServerWithCorrectSecurityLevel(jwtManager)
@@ -121,8 +121,10 @@ func startGrpc() error {
 	cpb.RegisterCsbiServiceServer(c.grpcServer, c.nbi.Csbi)
 	spb.RegisterSbiServiceServer(c.grpcServer, c.nbi.Sbi)
 	apb.RegisterAuthServiceServer(c.grpcServer, c.nbi.Auth)
+	apb.RegisterUserServiceServer(c.grpcServer, c.nbi.User)
+	apb.RegisterRoleServiceServer(c.grpcServer, c.nbi.Role)
 	go func() {
-		if err := c.grpcServer.Serve(lis); err != nil {
+		if err := c.grpcServer.Serve(lislisten); err != nil {
 			log.Fatal(err)
 		}
 	}()