Sean Harvey

Results 31 comments of Sean Harvey

This looks like it's done. There's some follow-up work I raised an issue for here: https://github.com/storj/edge/issues/569

Fixed in https://review.dev.storj.io/c/storj/minio/+/14433 and https://review.dev.storj.io/c/storj/minio/+/14434

All the supporting changes have been merged, the last one to do the validation is here: https://review.dev.storj.tools/c/storj/gateway/+/18834, then we can pass through the project created date in gateway-mt once we...

Additional unmapped or incorrect errors discovered in tests: https://review.dev.storj.io/c/storj/edge/+/14432 * `GetObjectRetention` on an object without retention unmapped error: `object retention not found: object does not have a retention configuration`. Maybe...

Jeremy mentioned that S3 returns `AccessDenied` if attempting to delete a locked object, so that one should probably keep the code as-is but provide a more useful message. S3 responds...

AWS S3 error response research: ### GetObjectLockConfiguration on bucket without object lock S3: HTTP 404: ObjectLockConfigurationNotFoundError: Object Lock configuration does not exist for this bucket Us: HTTP 400: "InvalidRequest: Bucket...

The current theory is retention times sent to S3 are truncated to seconds, but we might not be doing that. We might try `time.Truncate(time.Second)` on the requested retention expiration time...

https://raw.githubusercontent.com/aws/aws-cli/v2/CHANGELOG.rst ``` 2.23.0 ... * feature:``s3``: The S3 client attempts to validate response checksums for all S3 API operations that support checksums. However, if the SDK has not implemented the...

Artur and I discovered it changed in `2.23.0`. It works in `2.22.35`, which was the last version before it: https://github.com/aws/aws-cli/compare/2.22.35..2.23.0

Related: https://github.com/aws/aws-cli/issues/9214