nextcloud / files-clients

1 stars 0 forks source link

Use MultipartUpload for uploading chunks to s3 #28

Closed tobiasKaminsky closed 1 year ago

tobiasKaminsky commented 2 years ago

I will clarify client's side changes and then post more info.

tobiasKaminsky commented 2 years ago

To test:

Clarify if this is a benefit for iOS at all

juliusknorr commented 2 years ago

@tobiasKaminsky Sorry totally forgot to update you on the open points:

Regarding testing this if it is actually about a dev only setup you may use the minio play instance and just need to add this section to your config.php before installing Nextcloud, but files would then be uploaded to their public test instance:

Ref https://docs.min.io/minio/baremetal/console/minio-console.html

'objectstore' =>
        array (
            'class' => 'OC\\Files\\ObjectStore\\S3',
            'arguments' =>
            array (
                'bucket' => 'your-custom-bucket-name-maybe-random',
                'key' => 'Q3AM3UQ867SPQQA43P2F',
                'secret' => 'zuf+tfteSlswRu7BJ86wekitnifILbZam1KYY3TG',
                'hostname' => 'play.min.io',
                'port' => '9443',
                'use_ssl' => true,
                'use_path_style' => true,
                'autocreate' => true,
                'verify_bucket_exists' => true,
            ),
        )
juliusknorr commented 2 years ago

Further updates:

Full reference at https://github.com/nextcloud/server/pull/27034

tobiasKaminsky commented 1 year ago

5mb as minimum for every chunks, except last one.

schiessle commented 1 year ago

@tobiasKaminsky @juliushaertl sorry for digging out this old ticket. Does this also work if S3 is used as external storage or only with s3 primary storage?

juliusknorr commented 1 year ago

Only primary storage.