Follow top-level directory symlinks when finding resources

`filepath.Walk` doesn't follow symbolic links, so kubeconform reported
zero found files when pointed to a directory symlink. This adds support
to follow top-level links, nested links will still not be followed.
This commit is contained in:
Robert Rose 2025-06-11 15:38:33 +02:00
parent e65429b1e5
commit c691d7e07a

View file

@ -40,6 +40,24 @@ func isIgnored(path string, ignoreFilePatterns []string) (bool, error) {
return false, nil
}
func checkFile(path string, info os.FileInfo, files chan<- string, ignoreFilePatterns []string) error {
if !isYAMLFile(info) && !isJSONFile(info) {
return nil
}
ignored, err := isIgnored(path, ignoreFilePatterns)
if err != nil {
return err
}
if ignored {
return nil
}
files <- path
return nil
}
func findFilesInFolders(ctx context.Context, paths []string, ignoreFilePatterns []string) (chan string, chan error) {
files := make(chan string)
errors := make(chan error)
@ -59,21 +77,30 @@ func findFilesInFolders(ctx context.Context, paths []string, ignoreFilePatterns
return err
}
if !isYAMLFile(i) && !isJSONFile(i) {
return nil
// follow top-level directory symlinks
if i.Mode()&os.ModeSymlink != 0 {
linkInfo, err := os.Stat(path)
if err != nil {
return err
}
if linkInfo.IsDir() {
evalPath, err := filepath.EvalSymlinks(path)
if err != nil {
return err
}
filepath.Walk(evalPath, func(p string, i os.FileInfo, err error) error {
if err != nil {
return err
}
return checkFile(p, i, files, ignoreFilePatterns)
})
}
}
ignored, err := isIgnored(p, ignoreFilePatterns)
if err != nil {
return err
}
if ignored {
return nil
}
files <- p
return nil
return checkFile(p, i, files, ignoreFilePatterns)
})
if err != nil && err != io.EOF {