fix: Fix the problem of Docker restarting caused by proxy synchronization

Author: ssongliu

agent/app/api/v2/docker.go

@@ -72,7 +72,7 @@ func (b *BaseApi) UpdateDaemonJson(c *gin.Context) {
 		return
 	}
 
-	if err := dockerService.UpdateConf(req); err != nil {
+	if err := dockerService.UpdateConf(req, true); err != nil {
 		helper.InternalServer(c, err)
 		return
 	}

agent/app/service/docker.go

@@ -21,7 +21,7 @@ import (
 type DockerService struct{}
 
 type IDockerService interface {
-	UpdateConf(req dto.SettingUpdate) error
+	UpdateConf(req dto.SettingUpdate, withRestart bool) error
 	UpdateLogOption(req dto.LogOption) error
 	UpdateIpv6Option(req dto.Ipv6Option) error
 	UpdateConfByFile(info dto.DaemonJsonUpdateByFile) error
@@ -136,7 +136,7 @@ func (u *DockerService) LoadDockerConf() (*dto.DaemonJsonConf, error) {
 	return &data, nil
 }
 
-func (u *DockerService) UpdateConf(req dto.SettingUpdate) error {
+func (u *DockerService) UpdateConf(req dto.SettingUpdate, withRestart bool) error {
 	err := createIfNotExistDaemonJsonFile()
 	if err != nil {
 		return err
@@ -222,25 +222,31 @@ func (u *DockerService) UpdateConf(req dto.SettingUpdate) error {
 		}
 	}
 	if len(daemonMap) == 0 {
+		if len(file) == 0 {
+			return nil
+		}
 		_ = os.Remove(constant.DaemonJsonPath)
-		if err := restartDocker(); err != nil {
-			return err
+		if withRestart {
+			return restartDocker()
 		}
 		return nil
 	}
 	newJson, err := json.MarshalIndent(daemonMap, "", "\t")
 	if err != nil {
 		return err
 	}
+	if string(newJson) == string(file) {
+		return nil
+	}
 	if err := os.WriteFile(constant.DaemonJsonPath, newJson, 0640); err != nil {
 		return err
 	}
 	if err := validateDockerConfig(); err != nil {
 		return err
 	}
 
-	if err := restartDocker(); err != nil {
-		return err
+	if withRestart {
+		return restartDocker()
 	}
 	return nil
 }