fix: Fix the problem of failed MySQL permission modification (#8351)

Author: ssongliu

backend/app/service/cronjob_helper.go

@@ -350,12 +350,18 @@ func (u *CronjobService) removeExpiredBackup(cronjob model.Cronjob, accountMap m
 		if cronjob.Type == "snapshot" {
 			for _, account := range accounts {
 				if len(account) != 0 {
+					if _, ok := accountMap[account]; !ok {
+						continue
+					}
 					_, _ = accountMap[account].client.Delete(path.Join(accountMap[account].backupPath, "system_snapshot", records[i].FileName))
 				}
 			}
 			_ = snapshotRepo.Delete(commonRepo.WithByName(strings.TrimSuffix(records[i].FileName, ".tar.gz")))
 		} else {
 			for _, account := range accounts {
+				if _, ok := accountMap[account]; !ok {
+					continue
+				}
 				if len(account) != 0 {
 					_, _ = accountMap[account].client.Delete(path.Join(accountMap[account].backupPath, records[i].FileDir, records[i].FileName))
 				}

backend/utils/mysql/client/local.go

@@ -188,18 +188,19 @@ func (r *Local) ChangeAccess(info AccessChangeInfo) error {
 		info.Name = "*"
 		info.Password = r.Password
 	}
-	if info.Permission != info.OldPermission {
-		if err := r.Delete(DeleteInfo{
-			Version:     info.Version,
-			Username:    info.Username,
-			Permission:  info.OldPermission,
-			ForceDelete: true,
-			Timeout:     300}); err != nil {
-			return err
-		}
-		if info.Username == "root" {
-			return nil
-		}
+	if info.Permission == info.OldPermission {
+		return nil
+	}
+	if err := r.Delete(DeleteInfo{
+		Version:     info.Version,
+		Username:    info.Username,
+		Permission:  info.OldPermission,
+		ForceDelete: true,
+		Timeout:     300}); err != nil {
+		return err
+	}
+	if info.Username == "root" {
+		return nil
 	}
 	if err := r.CreateUser(CreateInfo{
 		Name:       info.Name,

backend/utils/mysql/client/remote.go

@@ -199,18 +199,19 @@ func (r *Remote) ChangeAccess(info AccessChangeInfo) error {
 		info.Name = "*"
 		info.Password = r.Password
 	}
-	if info.Permission != info.OldPermission {
-		if err := r.Delete(DeleteInfo{
-			Version:     info.Version,
-			Username:    info.Username,
-			Permission:  info.OldPermission,
-			ForceDelete: true,
-			Timeout:     300}); err != nil {
-			return err
-		}
-		if info.Username == "root" {
-			return nil
-		}
+	if info.Permission == info.OldPermission {
+		return nil
+	}
+	if err := r.Delete(DeleteInfo{
+		Version:     info.Version,
+		Username:    info.Username,
+		Permission:  info.OldPermission,
+		ForceDelete: true,
+		Timeout:     300}); err != nil {
+		return err
+	}
+	if info.Username == "root" {
+		return nil
 	}
 	if err := r.CreateUser(CreateInfo{
 		Name:       info.Name,

frontend/src/views/cronjob/index.vue

@@ -288,6 +288,7 @@ const onDelete = async (row: Cronjob.CronjobInfo | null) => {
     let ids = [];
     showClean.value = false;
     cleanData.value = false;
+    cleanRemoteData.value = true;
     if (row) {
         ids = [row.id];
         names = [row.name];