Congegni
/
sintonia
7
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

SAAS-922 - urlencode content headers to ensure compatibility

This commit is contained in:
Duncan Sommerville 2015-07-16 14:24:07 -04:00
parent 901b42c8c7
commit 041053b171
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
airtime_mvc/application/cloud_storage/Amazon_S3StorageBackend.php
View File

@ -46,7 +46,7 @@ class Amazon_S3StorageBackend extends StorageBackend
{ {
$urls = array(); $urls = array();
$s3args = array('ResponseContentDisposition' => 'attachment; filename="' . $contentDispositionFilename. '"'); $s3args = array('ResponseContentDisposition' => 'attachment; filename="' . urlencode($contentDispositionFilename) . '"');
$signedS3Url = $this->s3Client->getObjectUrl($this->getBucket(), $resourceId, '+60 minutes', $s3args); $signedS3Url = $this->s3Client->getObjectUrl($this->getBucket(), $resourceId, '+60 minutes', $s3args);
//If we're using the proxy cache, we need to modify the request URL after it has //If we're using the proxy cache, we need to modify the request URL after it has
@ -65,7 +65,7 @@ class Amazon_S3StorageBackend extends StorageBackend
$p["path"] = substr($p["path"], 1 + strlen($this->getBucket())); $p["path"] = substr($p["path"], 1 + strlen($this->getBucket()));
} }
$proxyUrl = $p["scheme"] . "://" . $p["host"] . $p["path"] . "?" . $p["query"]; $proxyUrl = $p["scheme"] . "://" . $p["host"] . $p["path"] . "?" . $p["query"];
//Add this proxy cache URL to the list of download URLs. //Add this proxy cache URL to the list of download URLs.s
array_push($urls, $proxyUrl); array_push($urls, $proxyUrl);
} }