fix: remove redundant logic

sparkyi · sparkyi · commit 88e44fb9fbc9 · 2026-02-09T14:25:34.000+08:00
diff --git a/internal/handler/handler.go b/internal/handler/handler.go
@@ -29,10 +29,8 @@ func copyFilesToTmpDir(tmpDir string, files map[string]string) error {
 			return fmt.Errorf("invalid file path %q: path traversal detected", relPath)
 		}
 
-		if strings.Contains(relPath, "/") {
-			if err = os.MkdirAll(filepath.Dir(targetPath), 0755); err != nil {
-				return err
-			}
+		if err = os.MkdirAll(filepath.Dir(targetPath), 0755); err != nil {
+			return err
 		}
 
 		if err = os.WriteFile(targetPath, []byte(content), 0644); err != nil {

Original file line number	Diff line number	Diff line change
`@@ -29,10 +29,8 @@ func copyFilesToTmpDir(tmpDir string, files map[string]string) error {`
`29`	`29`	`return fmt.Errorf("invalid file path %q: path traversal detected", relPath)`
`30`	`30`	`}`
`31`	`31`
`32`		`- if strings.Contains(relPath, "/") {`
`33`		`- if err = os.MkdirAll(filepath.Dir(targetPath), 0755); err != nil {`
`34`		`- return err`
`35`		`- }`
	`32`	`+ if err = os.MkdirAll(filepath.Dir(targetPath), 0755); err != nil {`
	`33`	`+ return err`
`36`	`34`	`}`
`37`	`35`
`38`	`36`	`if err = os.WriteFile(targetPath, []byte(content), 0644); err != nil {`