]> git-server-git.apps.pok.os.sepia.ceph.com Git - s3-tests.git/commit
sse s3 jumbo patch - all in one fixes for sse-s3 logic 440/head
authorPriya Sehgal <priya.sehgal@flipkart.com>
Fri, 30 Jul 2021 11:54:16 +0000 (17:24 +0530)
committerMarcus Watts <mwatts@redhat.com>
Fri, 29 Apr 2022 21:44:56 +0000 (17:44 -0400)
commitdd7cac25f54d5642f46d1a4a68603384c4fb3048
tree33be719cf905ea82e91fda5011919122dc5b8e07
parent101dfc104ab225c79aa60491a61d0ddf6d82c257
sse s3 jumbo patch - all in one fixes for sse-s3 logic

original tests by Priya Sehgal <priya.sehgal@flipkart.com>:
rgw/s3_boto3: Tests added for SSE-S3 (GET, PUT, HEAD, MPU).

Additions by Casey Bodley <cbodley@redhat.com>:
add 'sse-s3' tag to test cases
sse: add _put_bucket_encryption() helper function
sse: document test cases with default bucket encryption
sse: expects encryption response header on put/get
sse: add 8MB default-encrypted upload
sse: test uploads that request x-amz-server-side-encryption=AES256

Lastly all my changes (Marcus Watts <mwatts@redhat.com>):

remove obsolete test - do it only in boto3 now.

Combine or rename duplicated function names.
Giving more than one test the same name is a Bad Thing(tm).

sse: expand test_bucket_policy_put_obj_enc, and _put_bucket_encryption

test_bucket_policy_put_obj_enc was testing too many things at once.
new tests:

* customer encryption and sse-s3: should fail
* customer encryption and sse-kms: should fail
* deny if not sse-s3: no-enc fails, sse-s3 succeeds.
* deny if not sse-s3: kms fails
  deny if not sse-ksm: no-enc fails, sse-kms succeeds.
  deny if not sse-ksm: s3 fails

_put_bucket_encryption was only testing sse-s3.
* test both these variations: sse-s3 and sse-kms

Note:
* these tests will fail on pre-sse-s3 ceph.

python3: comment out all boto3.set_stream_logger() calls
They made too much output.

Signed-off-by: Marcus Watts <mwatts@redhat.com>
s3tests/functional/test_s3.py
s3tests_boto3/functional/test_s3.py