fogfish / stream

84%

main: 84%

Build Type

Pull #53

github

Committed by fogfish

Commit Message

(fix) Inconsistent file path behaviour

Key Changes:
Updated shared validation functions (filename.go):

IsValidFile(), IsValidPath(), and IsValidDir() now accept paths with or without leading /
Both /file.txt and file.txt are treated as equivalent paths relative to the mount point
Relative paths with .. are still rejected by fs.ValidPath()
Both filesystems now behave identically:

S3 filesystem (stream): Already had path normalization in s3Key() that strips leading /
Local filesystem (lfs): Uses trim() function to normalize paths before filesystem operations
Both accept the same path formats and treat them consistently
Updated documentation:

Both Create() and Open() methods in both filesystems now document that paths can be with or without leading /
Clear that both forms are treated as paths relative to the mount point
Behavior:
Before:

Paths required leading / → fsys.Create("/file.txt", nil) ✅
Paths without / failed → fsys.Create("file.txt", nil) ❌
After:

Both forms work equally → fsys.Create("/file.txt", nil) ✅
→ fsys.Create("file.txt", nil) ✅
Both are treated as the same file at the mount root
No support for relative traversal with ../ (correctly rejected)

Pull Request Pull Request #53: (fix) Inconsistent file path behaviour

Run Details

26 of 29 new or added lines in 2 files covered. (89.66%)

10 existing lines in 3 files now uncovered.

814 of 971 relevant lines covered (83.83%)

0.93 hits per line