mirror
/
afero
mirror of https://github.com/spf13/afero.git
1
0
Fork 1
Code Issues Packages Projects Releases Wiki Activity

allow absolute paths in BasePathFs when prefix is "" on windows

This commit is contained in:
DenKoren 2024-03-25 07:48:41 +01:00
parent 5c4385aa20
commit 08d1d4a126
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
basepath.go
View File

@ -51,7 +51,7 @@ func NewBasePathFs(source Fs, path string) Fs {
// on a file outside the base path it returns the given file name and an error, // on a file outside the base path it returns the given file name and an error,
// else the given file with the base path prepended // else the given file with the base path prepended
func (b *BasePathFs) RealPath(name string) (path string, err error) { func (b *BasePathFs) RealPath(name string) (path string, err error) {
if err := validateBasePathName(name); err != nil { if err := validateBasePathName(name, b.path); err != nil {
return name, err return name, err
} }
@ -64,7 +64,7 @@ func (b *BasePathFs) RealPath(name string) (path string, err error) {
return path, nil return path, nil
} }
func validateBasePathName(name string) error { func validateBasePathName(name string, base string) error {
if runtime.GOOS != "windows" { if runtime.GOOS != "windows" {
// Not much to do here; // Not much to do here;
// the virtual file paths all look absolute on *nix. // the virtual file paths all look absolute on *nix.
@ -73,7 +73,7 @@ func validateBasePathName(name string) error {
// On Windows a common mistake would be to provide an absolute OS path // On Windows a common mistake would be to provide an absolute OS path
// We could strip out the base part, but that would not be very portable. // We could strip out the base part, but that would not be very portable.
if filepath.IsAbs(name) { if filepath.IsAbs(name) && base != "" {
return os.ErrNotExist return os.ErrNotExist
} }