Fix `test_bucket_put_bad_canned_acl` ceph test #531

New issue

Open

opened 2024-11-01 12:45:05 +00:00 by dkirillov · 1 comment

dkirillov commented

2024-11-01 12:45:05 +00:00

Member

It would be nice the following tests pass:

test_bucket_put_bad_canned_acl

it expect 400 code if bucket canned acl is invalid. We return 500 now.

Describe the solution you'd like

Return 400 code

Describe alternatives you've considered

Additional context

## Is your feature request related to a problem? Please describe. It would be nice the following tests pass: * [test_bucket_put_bad_canned_acl](https://git.frostfs.info/TrueCloudLab/s3-tests/src/commit/d245097771698774d36a0283e9ef69bf67f3ba76/s3tests_boto3/functional/test_headers.py#L310) it expect 400 code if bucket canned acl is invalid. We return 500 now. ## Describe the solution you'd like Return 400 code ## Describe alternatives you've considered No ## Additional context No

alexvanin commented

2024-11-19 08:46:09 +00:00

Owner

I would like to see this change. It would be nice to provide a reason why s3 gateway throws 500. Seems like simple acl format checker solves an issue.