getNsMatch in eosfs should return a cleaned path (#5496)

jessegeens · web-flow · commit 332e9c5ae006 · 2026-02-03T16:18:32.000+01:00
diff --git a/changelog/unreleased/fix-get-ns-match.md b/changelog/unreleased/fix-get-ns-match.md
@@ -0,0 +1,3 @@
+Bugfix: getNsMatch in eosfs should return a cleaned path
+
+https://github.com/cs3org/reva/pull/5496
diff --git a/pkg/storage/utils/eosfs/eosfs.go b/pkg/storage/utils/eosfs/eosfs.go
@@ -319,7 +319,7 @@ func (fs *Eosfs) getNsMatch(internal string, nss []string) (string, error) {
 		return "", errtypes.NotFound(fmt.Sprintf("eosfs: path is outside namespaces: path=%s namespaces=%+v", internal, nss))
 	}
 
-	return match, nil
+	return path.Clean(match), nil
 }
 
 func (fs *Eosfs) unwrapInternal(ctx context.Context, ns, np string) (string, error) {

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	`+Bugfix: getNsMatch in eosfs should return a cleaned path`
	`2`	`+`
	`3`	`+https://github.com/cs3org/reva/pull/5496`
Original file line number	Diff line number	Diff line change
`@@ -319,7 +319,7 @@ func (fs *Eosfs) getNsMatch(internal string, nss []string) (string, error) {`
`319`	`319`	`return "", errtypes.NotFound(fmt.Sprintf("eosfs: path is outside namespaces: path=%s namespaces=%+v", internal, nss))`
`320`	`320`	`}`
`321`	`321`
`322`		`- return match, nil`
	`322`	`+ return path.Clean(match), nil`
`323`	`323`	`}`
`324`	`324`
`325`	`325`	`func (fs *Eosfs) unwrapInternal(ctx context.Context, ns, np string) (string, error) {`