update aws sdk and move it to 3rdparty
Signed-off-by: Robin Appelman <robin@icewind.nl>
This commit is contained in:
parent
14d94abf06
commit
f62e0a33f3
2
3rdparty
2
3rdparty
|
@ -1 +1 @@
|
|||
Subproject commit b7f163ff14791e747a0303314c6e17539adbc468
|
||||
Subproject commit a5e1de2925f81eeca18414c52c07d5e63020d7f7
|
|
@ -36,10 +36,6 @@
|
|||
|
||||
namespace OCA\Files_External\Lib\Storage;
|
||||
|
||||
set_include_path(get_include_path() . PATH_SEPARATOR .
|
||||
\OC_App::getAppPath('files_external') . '/3rdparty/aws-sdk-php');
|
||||
require_once 'aws-autoloader.php';
|
||||
|
||||
use Aws\S3\S3Client;
|
||||
use Aws\S3\Exception\S3Exception;
|
||||
use Icewind\Streams\CallbackWrapper;
|
||||
|
@ -230,13 +226,20 @@ class AmazonS3 extends \OC\Files\Storage\Common {
|
|||
|
||||
try {
|
||||
$files = array();
|
||||
$result = $this->getConnection()->getIterator('ListObjects', array(
|
||||
$results = $this->getConnection()->getPaginator('ListObjects', [
|
||||
'Bucket' => $this->bucket,
|
||||
'Delimiter' => '/',
|
||||
'Prefix' => $path
|
||||
), array('return_prefixes' => true));
|
||||
'Prefix' => $path,
|
||||
]);
|
||||
|
||||
foreach ($result as $object) {
|
||||
foreach ($results as $result) {
|
||||
// sub folders
|
||||
if (is_array($result['CommonPrefixes'])) {
|
||||
foreach ($result['CommonPrefixes'] as $prefix) {
|
||||
$files[] = substr(trim($prefix['Prefix'], '/'), strlen($path));
|
||||
}
|
||||
}
|
||||
foreach ($result['Contents'] as $object) {
|
||||
if (isset($object['Key']) && $object['Key'] === $path) {
|
||||
// it's the directory itself, skip
|
||||
continue;
|
||||
|
@ -246,6 +249,7 @@ class AmazonS3 extends \OC\Files\Storage\Common {
|
|||
);
|
||||
$files[] = $file;
|
||||
}
|
||||
}
|
||||
|
||||
return IteratorDirectory::wrap($files);
|
||||
} catch (S3Exception $e) {
|
||||
|
@ -270,7 +274,7 @@ class AmazonS3 extends \OC\Files\Storage\Common {
|
|||
));
|
||||
|
||||
$stat['size'] = $result['ContentLength'] ? $result['ContentLength'] : 0;
|
||||
if ($result['Metadata']['lastmodified']) {
|
||||
if (isset($result['Metadata']['lastmodified'])) {
|
||||
$stat['mtime'] = strtotime($result['Metadata']['lastmodified']);
|
||||
} else {
|
||||
$stat['mtime'] = strtotime($result['LastModified']);
|
||||
|
|
|
@ -22,11 +22,7 @@
|
|||
namespace OC\Files\ObjectStore;
|
||||
|
||||
use OCP\Files\ObjectStore\IObjectStore;
|
||||
|
||||
// TODO: proper composer
|
||||
set_include_path(get_include_path() . PATH_SEPARATOR .
|
||||
\OC_App::getAppPath('files_external') . '/3rdparty/aws-sdk-php');
|
||||
require_once 'aws-autoloader.php';
|
||||
use Psr\Http\Message\StreamInterface;
|
||||
|
||||
class S3 implements IObjectStore {
|
||||
use S3ConnectionTrait;
|
||||
|
@ -50,31 +46,17 @@ class S3 implements IObjectStore {
|
|||
* @since 7.0.0
|
||||
*/
|
||||
function readObject($urn) {
|
||||
// Create the command and serialize the request
|
||||
$request = $this->getConnection()->getCommand('GetObject', [
|
||||
$client = $this->getConnection();
|
||||
$command = $client->getCommand('GetObject', [
|
||||
'Bucket' => $this->bucket,
|
||||
'Key' => $urn
|
||||
])->prepare();
|
||||
]);
|
||||
$command['@http']['stream'] = true;
|
||||
$result = $client->execute($command);
|
||||
/** @var StreamInterface $body */
|
||||
$body = $result['Body'];
|
||||
|
||||
$request->dispatch('request.before_send', array(
|
||||
'request' => $request
|
||||
));
|
||||
|
||||
$headers = $request->getHeaderLines();
|
||||
$headers[] = 'Connection: close';
|
||||
|
||||
$opts = [
|
||||
'http' => [
|
||||
'method' => "GET",
|
||||
'header' => $headers
|
||||
],
|
||||
'ssl' => [
|
||||
'verify_peer' => true
|
||||
]
|
||||
];
|
||||
|
||||
$context = stream_context_create($opts);
|
||||
return fopen($request->getUrl(), 'r', false, $context);
|
||||
return $body->detach();
|
||||
}
|
||||
|
||||
/**
|
||||
|
|
|
@ -53,7 +53,7 @@ trait S3ConnectionTrait {
|
|||
$this->bucket = $params['bucket'];
|
||||
$this->timeout = (!isset($params['timeout'])) ? 15 : $params['timeout'];
|
||||
$params['region'] = empty($params['region']) ? 'eu-west-1' : $params['region'];
|
||||
$params['hostname'] = empty($params['hostname']) ? 's3.amazonaws.com' : $params['hostname'];
|
||||
$params['hostname'] = empty($params['hostname']) ? 's3.' . $params['region'] . '.amazonaws.com' : $params['hostname'];
|
||||
if (!isset($params['port']) || $params['port'] === '') {
|
||||
$params['port'] = (isset($params['use_ssl']) && $params['use_ssl'] === false) ? 80 : 443;
|
||||
}
|
||||
|
@ -76,20 +76,23 @@ trait S3ConnectionTrait {
|
|||
$base_url = $scheme . '://' . $this->params['hostname'] . ':' . $this->params['port'] . '/';
|
||||
|
||||
$options = [
|
||||
'version' => isset($this->params['version']) ? $this->params['version'] : 'latest',
|
||||
'credentials' => [
|
||||
'key' => $this->params['key'],
|
||||
'secret' => $this->params['secret'],
|
||||
'base_url' => $base_url,
|
||||
],
|
||||
'endpoint' => $base_url,
|
||||
'region' => $this->params['region'],
|
||||
S3Client::COMMAND_PARAMS => [
|
||||
'command.params' => [
|
||||
'PathStyle' => isset($this->params['use_path_style']) ? $this->params['use_path_style'] : false,
|
||||
]
|
||||
];
|
||||
if (isset($this->params['proxy'])) {
|
||||
$options[S3Client::REQUEST_OPTIONS] = ['proxy' => $this->params['proxy']];
|
||||
$options['request.options'] = ['proxy' => $this->params['proxy']];
|
||||
}
|
||||
$this->connection = S3Client::factory($options);
|
||||
$this->connection = new S3Client($options);
|
||||
|
||||
if (!$this->connection->isValidBucketName($this->bucket)) {
|
||||
if (!S3Client::isBucketDnsCompatible($this->bucket)) {
|
||||
throw new \Exception("The configured bucket name is invalid.");
|
||||
}
|
||||
|
||||
|
@ -98,11 +101,6 @@ trait S3ConnectionTrait {
|
|||
$this->connection->createBucket(array(
|
||||
'Bucket' => $this->bucket
|
||||
));
|
||||
$this->connection->waitUntilBucketExists(array(
|
||||
'Bucket' => $this->bucket,
|
||||
'waiter.interval' => 1,
|
||||
'waiter.max_attempts' => 15
|
||||
));
|
||||
$this->testTimeout();
|
||||
} catch (S3Exception $e) {
|
||||
\OCP\Util::logException('files_external', $e);
|
||||
|
|
Loading…
Reference in New Issue