Skip to content

Legacy client constructor fix#3547

Merged
sbaluja merged 3 commits intomainfrom
LegacyClientConstructorFix
Sep 17, 2025
Merged

Legacy client constructor fix#3547
sbaluja merged 3 commits intomainfrom
LegacyClientConstructorFix

Conversation

@sbaluja
Copy link
Contributor

@sbaluja sbaluja commented Sep 17, 2025

Issue #, if available:

Description of changes:

  • Update legacy client constructors to use the CredentialProviderConfiguration struct
  • Update tests that explicitly declare profile name to not do that since profile name actually works now

Check all that applies:

  • Did a review by yourself.
  • Added proper tests to cover this PR. (If tests are not applicable, explain.)
  • Checked if this PR is a breaking (APIs have been changed) change.
  • Checked if this PR will not introduce cross-platform inconsistent behavior.
  • Checked if this PR would require a ReadMe/Wiki update.

Check which platforms you have built SDK on to verify the correctness of this PR.

  • Linux
  • Windows
  • Android
  • MacOS
  • IOS
  • Other Platforms

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@sbaluja sbaluja force-pushed the LegacyClientConstructorFix branch from 0d15ded to 83f796a Compare September 17, 2025 14:59
@sbaluja sbaluja merged commit fbc777b into main Sep 17, 2025
4 of 6 checks passed
@sbaluja sbaluja deleted the LegacyClientConstructorFix branch October 27, 2025 17:14
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.

4 participants