Skip to content

[S3] Use Authorization header instead of query param in S3 XML in/out#1640

Merged
cb-spider merged 1 commit intocloud-barista:masterfrom
powerkimhub:refactor/s3-xmlformat-test-query-to-auth-header
Jan 6, 2026
Merged

[S3] Use Authorization header instead of query param in S3 XML in/out#1640
cb-spider merged 1 commit intocloud-barista:masterfrom
powerkimhub:refactor/s3-xmlformat-test-query-to-auth-header

Conversation

@powerkimhub
Copy link
Copy Markdown
Member

@powerkimhub powerkimhub commented Jan 6, 2026

Summary

Migrates S3 XML format test scripts from query parameter authentication to Authorization header method.

Key Changes

  • Authentication Method: Update test scripts to replace ?ConnectionName= query param with Authorization: AWS4-HMAC-SHA256 Credential=$CONNECTION_NAME header in all curl requests
  • Cleanup Order: Add explicit multipart upload and object cleanup before bucket deletion

Benefits

  • Aligned with API authentication best practices
  • More robust test cleanup preventing orphaned resources

Test Results

====================================================================================
                         S3 API TEST SUMMARY - ALL CSPs
====================================================================================

CSP          | Bucket   | Object   | Multipart  | Versioning | CORS     | Special  | Total
------------------------------------------------------------------------------------
AWS          | 6/6      | 6/6      | 6/6        | 4/4        | 4/4      | 6/6      | 32/32
GCP          | 6/6      | 6/6      | 6/6        | 4/4        | 4/4      | 6/6      | 32/32
ALIBABA      | 6/6      | 6/6      | 6/6        | 4/4        | 4/4      | 6/6      | 32/32
TENCENT      | 6/6      | 6/6      | 6/6        | 4/4        | 4/4      | 6/6      | 32/32
IBM          | 6/6      | 6/6      | 6/6        | 4/4        | 4/4      | 6/6      | 32/32
OPENSTACK    | 6/6      | 6/6      | 0/6 (NA)   | 0/4 (NA)   | 4/4      | 6/6      | 22/22
NCP          | 6/6      | 6/6      | 6/6        | 0/4 (NA)   | 0/4 (NA) | 6/6      | 24/24
NHN          | 6/6      | 6/6      | 6/6        | 0/4 (NA)   | 0/4 (NA) | 6/6      | 24/24
KT           | 6/6      | 6/6      | 6/6        | 4/4        | 4/4      | 6/6      | 32/32
------------------------------------------------------------------------------------

Legend: X/Y (Pass/Total), NA = Not Applicable (CSP does not support)

Test Categories:
  - Bucket      : Bucket management operations (6 tests)
  - Object      : Object operations (6 tests)
  - Multipart   : Multipart upload (6 tests)
  - Versioning  : Bucket versioning (4 tests)
  - CORS        : CORS configuration (4 tests)
  - Special     : PreSigned URLs & Force operations (6 tests)

====================================================================================

@powerkimhub powerkimhub requested a review from cb-spider January 6, 2026 08:10
@cb-spider cb-spider merged commit de264c8 into cloud-barista:master Jan 6, 2026
2 checks passed
@powerkimhub powerkimhub deleted the refactor/s3-xmlformat-test-query-to-auth-header branch January 6, 2026 08:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cb-spider cb-spider cb-spider approved these changes

Assignees

No one assigned

Labels

test

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@powerkimhub @cb-spider