archtechx/tenancy
1
0
Fork 0
mirror of https://github.com/archtechx/tenancy.git synced 2026-09-20 14:14:03 +00:00
Code Activity

Add comment above $attemptedPath

Browse source 
At first glance, it could look weird that $attemptedPath uses "/" but the check in abortIf below uses DIRECTORY_SEPARATOR. Add comment that explains this.
This commit is contained in:
lukinovec 2026-09-07 10:42:18 +02:00 • committed by Samuel Stancl
parent efc0877f10
commit e7c0193931
1 changed files with 1 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

1
src/Controllers/TenantAssetController.php
View file

@ -142,6 +142,7 @@ class TenantAssetController implements HasMiddleware
// The asset root doesn't exist, so it cannot contain files // The asset root doesn't exist, so it cannot contain files
$this->abortIf($allowedRoot === false, "Storage root doesn't exist"); $this->abortIf($allowedRoot === false, "Storage root doesn't exist");
// realpath() ensures the directory exists and converts / to \ on Windows
$attemptedPath = realpath("{$allowedRoot}/{$path}"); $attemptedPath = realpath("{$allowedRoot}/{$path}");
// User is attempting to access a nonexistent file // User is attempting to access a nonexistent file