-
Notifications
You must be signed in to change notification settings - Fork 175
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
PLAT-10892: return s3 file location or local server file path for path only requests coming from s3 supported packager #9419
Conversation
…h only requests coming from s3 supported apckager
… if all flavor exist locally
{ | ||
// if the entry fully exist locally return empty fileSync and path | ||
// to make the request redirected for local serving | ||
if(self::entryFullyExistLocally($flavorAsset->getEntryId())) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
check the opposite that none exist in the $preferredStorageId
else | ||
{ | ||
// use cloud file location | ||
$path = self::CLOUD_S3_PREFIX . $fileSync->getFilePath(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Try to get the prefix from the remote storage
* @return string | ||
* @throws Exception | ||
*/ | ||
public static function getPathByFileSync($fileSync) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If the preferred storage ID is not local - user URL else user file.
{ | ||
$allowRemotePath = false; | ||
if(!$local) | ||
$file_sync = kFileSyncUtils::getFileSyncByPreferredOrder($syncKey, $flavorAsset, $preferredStorageId); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rename - no Order
This change is