Golang WalkDir结合Chmod递归修改权限仅首个路径生效问题

你的代码运行到第一个路径就停止，核心是回调函数的错误处理逻辑完全不符合filepath.WalkDir的设计规则，具体问题如下：

• WalkDir的回调函数只要返回非nil错误，就会立刻终止整个遍历流程。你在回调里只要遇到错误就直接panic或者log.Fatal，会直接中断整个程序，自然不会继续处理后续文件。
• 错误变量混用：getFileMode、os.Chmod返回的错误存在err2里，出错时你panic的却是WalkDir传入的err参数，这个参数和当前操作的错误完全无关，不仅错误信息无效，甚至可能出现panic(nil)的情况。
• 重复做无意义的初始化：每次处理一个路径都要读环境变量、解析一次权限模式，这类操作只需要在遍历开始前做一次就够，放在回调里如果解析失败，第一个路径就会直接触发崩溃退出。
• 完全忽略WalkDir传入的err参数：当WalkDir访问某个路径遇到权限不足、路径不存在等问题时，会把错误通过第三个参数传给回调，你直接跳过这个判断去执行Chmod，很容易触发无效路径的操作错误。

func ChmodRec(fileMode os.FileMode) fs.WalkDirFunc {
    // 闭包持有提前解析好的权限模式，遍历过程中直接使用
    return func(path string, di fs.DirEntry, err error) error {
        // 先处理WalkDir上报的路径访问错误
        if err != nil {
            fmt.Printf("Skip path %s, access failed: %v\n", path, err)
            // 要跳过错误继续遍历就返回nil，要终止遍历就返回err
            return nil
        }

        // 修改权限
        if err := os.Chmod(path, fileMode); err != nil {
            fmt.Printf("Chmod %s failed: %v\n", path, err)
            // 单个文件修改失败不中断整体遍历
            return nil
        }
        fmt.Printf("Changed permission of %s to %v\n", path, fileMode)
        return nil
    }
}

func ChmodRecursive(path string, mode string) error {
    // 遍历开始前一次性解析权限模式，失败直接返回
    fileMode, err := getFileMode(mode)
    if err != nil {
        return fmt.Errorf("parse mode %s failed: %w", mode, err)
    }

    // 启动遍历
    if err := filepath.WalkDir(path, ChmodRec(fileMode)); err != nil {
        return fmt.Errorf("walk path %s failed: %w", path, err)
    }
    return nil
}

func main() {
    targetPath := "bla/test/"
    targetMode := "755"
    if err := ChmodRecursive(targetPath, targetMode); err != nil {
        log.Fatalf("Recursive chmod failed: %v", err)
    }
}

• 不要在WalkDir的回调里滥用panic、log.Fatal这类会直接终止程序的调用，非致命错误（单个文件访问/修改失败）直接返回nil跳过即可，只有遇到必须终止遍历的错误才返回非nil值。
• 所有遍历前就能完成的初始化操作（比如解析权限模式、校验输入参数）不要放到回调里执行，既影响性能，也会导致遍历刚启动就因为初始化错误中断。
• 回调的第三个err参数必须优先处理，不要跳过判断直接操作路径，避免触发空指针、无效路径这类低级崩溃。
• 去掉原来用环境变量传权限模式的逻辑，用闭包或者函数传参的方式传递配置更可靠，不会出现环境变量被意外修改的问题。

内容的提问来源于stack exchange，提问作者starryn1ght