Support "HEAD Bucket" request - Githubissues

leo-project / leofs

The LeoFS Storage System

https://leo-project.net/leofs/

Apache License 2.0

1.55k stars 155 forks source link

Support "HEAD Bucket" request #90

Closed Matsue closed 11 years ago

Matsue commented 11 years ago

I could not get expect values by doesBucketExist method on AWS-SDK-PHP. LeoFS looks does not support "HEAD Bucket" request.

the error was occured here https://github.com/owncloud/core/blob/master/apps/files_external/lib/amazons3.php#L102
AWS-SDK-PHP document http://docs.aws.amazon.com/aws-sdk-php-2/latest/class-Aws.S3.S3Client.html#_doesBucketExist
AWS-SDK-PHP sources https://github.com/aws/aws-sdk-php/blob/master/src/Aws/S3/S3… https://github.com/aws/aws-sdk-php/blob/master/src/Aws/S3/Re…
HEAD Bucket request document http://docs.aws.amazon.com/AmazonS3/latest/API/RESTBucketHEAD.html

yosukehara commented 11 years ago

Thank you for your report. We will support it with next version.

mocchira commented 11 years ago

@Matsue please confirtm whether your problem is solved?

mocchira commented 11 years ago

I confirmed that this issue still exist, so reopened

mocchira commented 11 years ago

It turned out that current implementation doesn't handle subresource such as ?versioning, ?acl etc.. correctly. Since the latest aws-sdk used subresource when invoking doesBucketExist , this issue occured. We will fix this issue ASAP.

mocchira commented 11 years ago

Done temporary solution here. https://github.com/leo-project/leo_s3_libs/commit/6e3de47f4311795785a3a8cf22069fc9b2c383be

We will fix fundamentally next week.

mocchira commented 11 years ago

@Matsue please confirtm if this problem is solved in your env?