Merge pull request #21317 from FlorentCoppint/s3_upload_part_size

Upload part size as S3 parameter instead of constant value

Author: rullzer

lib/private/Files/ObjectStore/S3ConnectionTrait.php

@@ -47,6 +47,9 @@ trait S3ConnectionTrait {
 
 	/** @var int */
 	protected $timeout;
+	
+	/** @var int */
+	protected $uploadPartSize;
 
 	protected $test;
 
@@ -60,6 +63,7 @@ protected function parseParams($params) {
 		$this->test = isset($params['test']);
 		$this->bucket = $params['bucket'];
 		$this->timeout = !isset($params['timeout']) ? 15 : $params['timeout'];
+		$this->uploadPartSize = !isset($params['uploadPartSize']) ? 524288000 : $params['uploadPartSize'];
 		$params['region'] = empty($params['region']) ? 'eu-west-1' : $params['region'];
 		$params['hostname'] = empty($params['hostname']) ? 's3.' . $params['region'] . '.amazonaws.com' : $params['hostname'];
 		if (!isset($params['port']) || $params['port'] === '') {

lib/private/Files/ObjectStore/S3ObjectTrait.php

@@ -33,8 +33,6 @@
 use Icewind\Streams\CallbackWrapper;
 use OC\Files\Stream\SeekableHttpStream;
 
-const S3_UPLOAD_PART_SIZE = 524288000; // 500MB
-
 trait S3ObjectTrait {
 	/**
 	 * Returns the connection
@@ -91,7 +89,7 @@ public function writeObject($urn, $stream) {
 		$uploader = new MultipartUploader($this->getConnection(), $countStream, [
 			'bucket' => $this->bucket,
 			'key' => $urn,
-			'part_size' => S3_UPLOAD_PART_SIZE,
+			'part_size' => $this->uploadPartSize,
 		]);
 
 		try {