Skip to content

S3 supported object name validation coverage - #899

Merged
mergify[bot] merged 1 commit into
red-hat-storage:masterfrom
ckulal:object_name_validation2
Aug 29, 2026
Merged

mergify[bot] merged 1 commit into
red-hat-storage:masterfrom
ckulal:object_name_validation2

Conversation

@ckulal

@ckulal ckulal commented Aug 23, 2026

Copy link
Copy Markdown
Contributor

S3 supported object name validation coverage:

Adds RGW coverage for S3 object key naming against the 
Amazon S3 object key guidelines.

The test creates an RGW user and bucket, then PUT/GET/DELETE objects whose keys exercise each AWS category. 

Categories

  • Safe characters — alphanumerics, ! - _ . * ' ( ), documented AWS examples, prefix/delimiter keys, trailing period, and case-sensitive names

  • Period-only path segments — . / .. segments that remain valid via the S3 API (./file.txt, folder/../file.txt, hidden/backup names, AWS valid relative path videos/2014/../../video1.wmv, key .)


  • Special handling — & $ @ = ; / : + space , ?, leading/trailing/consecutive slashes, non-ASCII UTF-8, and ASCII 00–1F / 7F (NUL is expected to fail)


  • Characters to avoid — \ { } ^ % \ ] " > [ ~ < # |` and ASCII 128–255; still expected to succeed over the API


  • Hard limits — empty key (reject), 1024-byte UTF-8 max (accept), 1025 bytes (reject), multibyte UTF-8 at/over the limit, and key soap (allowed with path-style)


  • Confirm safe, period-segment, special-handling, and avoid keys PUT/GET successfully

  • Confirm empty key, keys over 1024 UTF-8 bytes, and NUL are rejected

  • Confirm objects and the test bucket are cleaned up

Pass log: http://magna002.ceph.redhat.com/ceph-qe-logs/obj_name_validation/test_object_name_validation.console.log2

Signed-off-by: chaithra <ckulal@redhat.com>
@ckulal
ckulal force-pushed the object_name_validation2 branch from 6562fb7 to 1e673ea Compare August 23, 2026 06:04
@ckulal
ckulal requested a review from a team August 23, 2026 09:03
@TejasC88

Copy link
Copy Markdown
Contributor

LGTM

@mergify
mergify Bot merged commit 3c2df08 into red-hat-storage:master Aug 29, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants