linting

controller/controller.go

@@ -448,7 +448,7 @@ func ensureDefaultUserExists() error {
 
 func deletAllExpiredUserTokens() error {
 	var usersToUpdate []rbac.User
-	// Temporarly create JWT manager just to evaluate tokens here
+	// Temporary create JWT manager just to evaluate tokens here
 	jwtManager := rbacImpl.NewJWTManager(config.JWTSecret, config.JWTDuration)
 
 	users, err := c.userService.GetAll()
@@ -480,7 +480,6 @@ func deletAllExpiredUserTokens() error {
 		}
 	}
 	return nil
-
 }
 
 // Run calls initialize to start the controller.

controller/northbound/server/auth.go

@@ -95,7 +95,10 @@ func (s AuthServer) Login(ctx context.Context, request *apb.LoginRequest) (*apb.
 
 	userToUpdate.AddToken(token)
 	for len(userToUpdate.GetTokens()) > config.MaxTokensPerUser {
-		userToUpdate.RemoveToken(userToUpdate.GetTokens()[0])
+		err = userToUpdate.RemoveToken(userToUpdate.GetTokens()[0])
+		if err != nil {
+			return nil, err
+		}
 	}
 	err = s.userService.Update(userToUpdate)
 	if err != nil {