add user-remove command

cli/cmd/commands.go

@@ -504,6 +504,7 @@ func newNodeCommand() *cobra.Command {
 		Hidden: true,
 	}
 	addCommand(cmd, newNodeUserAddCommand())
+	addCommand(cmd, newNodeUserRemoveCommand())
 	addCommand(cmd, newNodeLoginCommand())
 	addCommand(cmd, newNodeLogoutCommand())
 	return cmd
@@ -524,6 +525,21 @@ func newNodeUserAddCommand() *cobra.Command {
 	return cmd
 }
 
+func newNodeUserRemoveCommand() *cobra.Command {
+	var a controller.NodeUserRemoveArgs
+	cmd := &cobra.Command{
+		Use:  "user-remove",
+		Args: cobra.ExactArgs(1),
+		RunE: func(cmd *cobra.Command, args []string) error {
+			a.Username = args[0]
+			return controller.NodeUserRemove(a)
+		},
+	}
+	setUsageTemplate(cmd, texts.Deploy.Arguments)
+	cmd.Flags().StringVarP(&a.Node, "node", "n", "", "Node in which the user will be removed")
+	return cmd
+}
+
 func newNodeLoginCommand() *cobra.Command {
 	var a controller.NodeLoginArgs
 	cmd := &cobra.Command{

cli/controller/node.go

@@ -8,8 +8,9 @@ import (
 )
 
 const (
-	UserAddHTTPMethod = "node/addUser"
-	LoginHTTPMethod   = "auth/login"
+	UserAddHTTPMethod    = "node/addUser"
+	UserRemoveHTTPMethod = "node/removeUser"
+	LoginHTTPMethod      = "auth/login"
 )
 
 type NodeUserAddArgs struct {
@@ -37,6 +38,25 @@ func NodeUserAdd(a NodeUserAddArgs) error {
 	}, nil)
 }
 
+type NodeUserRemoveArgs struct {
+	Node     string
+	Username string
+}
+
+func NodeUserRemove(a NodeUserRemoveArgs) error {
+	n, err := findNode(a.Node)
+	if err != nil {
+		return err
+	}
+	i, err := nodeInvoker(n)
+	if err != nil {
+		return err
+	}
+	return i.Do(UserRemoveHTTPMethod, &dto.RemoveUserRequest{
+		Username: a.Username,
+	}, nil)
+}
+
 func findNode(name string) (*domain.Node, error) {
 	fs, err := domain.NewSingleDeveloperWorkspaceStore()
 	if err != nil {

node/api/node/store.go

@@ -41,6 +41,10 @@ func (s *Store) StoreUser(name string, role domain.Role) error {
 	})
 }
 
+func (s *Store) RemoveUser(name string) error {
+	return s.users.Delete(name)
+}
+
 func (s *Store) FindUser(name string) (*user, error) {
 	u := &user{}
 	if err := s.users.Get(name, u); err != nil {

node/dto/dto.go

@@ -121,6 +121,11 @@ type AddUserRequest struct {
 	Role     domain.Role
 }
 
+type RemoveUserRequest struct {
+	Username string
+	Role     domain.Role
+}
+
 type LoginResponse struct {
 	Node *domain.Node
 }

node/functions/node/main.go

@@ -35,6 +35,17 @@ func (n *Node) AddUser(ctx context.Context, req *dto.AddUserRequest) error {
 	return n.store.StoreUser(req.Username, req.Role)
 }
 
+func (n *Node) RemoveUser(ctx context.Context, req *dto.RemoveUserRequest) error {
+	ok, err := domain.IsAdmin(ctx)
+	if err != nil {
+		return err
+	}
+	if !ok {
+		return domain.ErrNotAuthorized
+	}
+	return n.store.RemoveUser(req.Username)
+}
+
 func main() {
 	var api = New()
 	mantil.LambdaHandler(api)