Skip to content

Conversation

kuhe
Copy link
Contributor

@kuhe kuhe commented Aug 8, 2025

Issue

smithy-lang/smithy-typescript#1600

Description

For schema-serde mode:

  • applies model transform removing the /{Bucket} endpoint uri prefix IFF input members group has Bucket and the member is a contextParam (trait).
  • error handlers for schema-serde protocols have more consistent error metadata assignment

Testing

S3 e2e tests, integration (CI)

Checklist

  • [n/a] If the PR is a feature, add integration tests (*.integ.spec.ts).
  • [n/a] If you wrote E2E tests, are they resilient to concurrent I/O?
  • [n/a] If adding new public functions, did you add the @public tag and enable doc generation on the package?

@kuhe kuhe requested a review from a team as a code owner August 8, 2025 16:17
// Bucket is in the http trait but is handled by endpoints ruleset.
return segment !== "{Bucket}";
})
.join("/") || "/";
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

removed this hack, replaced by model preprocessing transform

@kuhe kuhe force-pushed the schema/logger branch 3 times, most recently from cfcc887 to 6207b9d Compare August 8, 2025 16:34
@kuhe kuhe force-pushed the schema/logger branch 4 times, most recently from 5dc79a4 to 2448107 Compare August 8, 2025 19:50
@kuhe kuhe changed the title chore(client-s3): bucket contextParam customization for schema-serde mode chore(middleware-logger): use log-filter with schema when available Aug 26, 2025
@kuhe kuhe changed the title chore(middleware-logger): use log-filter with schema when available chore(client-s3): bucket contextParam customization for schema-serde mode Aug 26, 2025
@kuhe kuhe merged commit 933c5c0 into aws:main Aug 26, 2025
3 checks passed
@kuhe kuhe deleted the schema/logger branch August 26, 2025 16:42
Copy link

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs and link to relevant comments in this thread.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 10, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants