refactor: use maps (#10154)

agent/app/service/app_utils.go

@@ -7,6 +7,7 @@ import (
 	"encoding/json"
 	"fmt"
 	"log"
+	"maps"
 	"math"
 	"net/http"
 	"os"
@@ -1044,11 +1045,11 @@ func copyData(task *task.Task, app model.App, appDetail model.AppDetail, appInst
 		return
 	}
 	envPath := path.Join(appDir, ".env")
-	envParams := make(map[string]string, len(req.Params))
+	var envParams map[string]string
 	if fileOp.Stat(envPath) {
 		envs, _ := gotenv.Read(envPath)
-		for k, v := range envs {
+		if envParams = maps.Clone(envs); envParams == nil {
-			envParams[k] = v
+			envParams = make(map[string]string)
 		}
 	}
 	handleMap(req.Params, envParams)

agent/app/service/runtime.go

@@ -6,6 +6,7 @@ import (
 	"encoding/json"
 	"fmt"
 	fcgiclient "github.com/tomasen/fcgi_client"
+	"maps"
 	"os"
 	"path"
 	"path/filepath"
@@ -1092,9 +1093,7 @@ func (r *RuntimeService) UpdatePHPContainer(req request.PHPContainerConfig) erro
 	}
 	newMap := make(map[string]string)
 	handleMap(create.Params, newMap)
-	for k, v := range newMap {
+	maps.Copy(envs, newMap)
-		envs[k] = v
-	}
 	envs["PANEL_APP_PORT_HTTP"] = runtime.Port
 	envStr, err := gotenv.Marshal(envs)
 	if err != nil {

agent/app/service/runtime_utils.go

@@ -7,6 +7,7 @@ import (
 	"encoding/json"
 	"fmt"
 	"io"
+	"maps"
 	"os"
 	"os/exec"
 	"path"
@@ -522,9 +523,7 @@ func handleParams(create request.RuntimeCreate, projectDir string) (composeConte
 
 	newMap := make(map[string]string)
 	handleMap(create.Params, newMap)
-	for k, v := range newMap {
+	maps.Copy(env, newMap)
-		env[k] = v
-	}
 
 	envStr, err := gotenv.Marshal(env)
 	if err != nil {

agent/utils/cloud_storage/client/helper/webdav/auth_digest.go

@@ -6,6 +6,7 @@ import (
 	"encoding/hex"
 	"fmt"
 	"io"
+	"maps"
 	"net/http"
 	"strings"
 )
@@ -46,9 +47,9 @@ func (d *DigestAuth) Close() error {
 }
 
 func (d *DigestAuth) Clone() Authenticator {
-	parts := make(map[string]string, len(d.digestParts))
+	var parts map[string]string
-	for k, v := range d.digestParts {
+	if parts = maps.Clone(parts); parts == nil {
-		parts[k] = v
+		parts = make(map[string]string)
 	}
 	return &DigestAuth{user: d.user, pw: d.pw, digestParts: parts}
 }