Skip to content

Conversation

@dayaffe
Copy link
Contributor

@dayaffe dayaffe commented Jul 11, 2025

Breaking Change: Removed authSchemePreference from the default auth scheme resolver initializer.
Bug Fix: authSchemePreference will now operate as expected when passed into the service config

Issue #

Description of changes

New/existing dependencies impact assessment, if applicable

Conventional Commits

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

val context = setupTests("http-binding-protocol-generator-test.smithy", "com.test#Example")
val contents = getClientFileContents("Sources/Example", "ExampleClient.swift", context.manifest)
contents.shouldSyntacticSanityCheck()
print(contents)
Copy link
Contributor

Choose a reason for hiding this comment

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

remove print statement

Copy link
Contributor Author

Choose a reason for hiding this comment

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

whoops

@jbelkins jbelkins self-requested a review July 11, 2025 21:03
@jbelkins jbelkins changed the title fix: authSchemePreference in config + remove from default resolver fix!: authSchemePreference in config + remove from default resolver Jul 14, 2025
@jbelkins jbelkins merged commit e0a357e into main Jul 14, 2025
31 checks passed
@jbelkins jbelkins deleted the day/fix-auth-scheme-pref branch July 14, 2025 18:07
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