Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor endpoint bindings/middleware #455

Merged
merged 5 commits into from
Sep 29, 2023

Conversation

lucix-aws
Copy link
Contributor

Refactor endpoint parameter bindings and middleware:

  • A single function bindEndpointParams now handles bindings universally, which will allow us to reuse for auth in s3/eventbridge
    • bindEndpointParams delegates to the operation input struct to handle contextParam staticContextParam bindings.
  • Refactored how builtin bindings are exposed to be more streamlined, they're now registered in GoIntegration.
  • Refactored endpoint v2 resolution to be a single middleware per service.
  • Introduced SmithyGoTypes and GoStdlibTypes symbol constants to remove the need to duplicate calls to SymbolUtils.createValueSymbolBuilder(name[, dep]).build() everywhere

@lucix-aws lucix-aws merged commit 7462ce1 into feat-sraauth Sep 29, 2023
1 check passed
@lucix-aws lucix-aws deleted the feat-sraauth-endpointbindings branch September 29, 2023 14:50
lucix-aws added a commit that referenced this pull request Oct 2, 2023
lucix-aws added a commit that referenced this pull request Oct 2, 2023
lucix-aws added a commit that referenced this pull request Oct 16, 2023
lucix-aws added a commit that referenced this pull request Oct 16, 2023
lucix-aws added a commit that referenced this pull request Oct 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants