Skip to content

Support Microsoft Entra auth in Data Transfer connections - #106

Merged
alex-bochkov merged 3 commits into
mainfrom
codex/validate-data-transfer-window-support
May 17, 2026
Merged

Support Microsoft Entra auth in Data Transfer connections#106
alex-bochkov merged 3 commits into
mainfrom
codex/validate-data-transfer-window-support

Conversation

@alex-bochkov

Copy link
Copy Markdown
Contributor

Motivation

  • Ensure the Data Transfer UI and related connection helpers honor Microsoft Entra / Azure AD authentication modes so interactive/MFA and other Entra flows work when selecting source/target SQL Server connections.

Description

  • Switched SQL client using directive in the Data Transfer window to Microsoft.Data.SqlClient to allow Entra-related connection-string/authentication features. (file: DataTransfer/DataTransferWindowControl.xaml.cs)
  • Centralized and extended authentication handling in ScriptFactoryAccess by adding reflection-based extraction of authentication metadata and helper functions: GetAuthenticationMode(object), AddPropertyValue, ApplyAuthentication, GetSqlAuthenticationMethod, ContainsAny, ShouldSendPassword, and an updated interactive-detection ShouldUseMicrosoftEntraInteractive to decide when to use SqlAuthenticationMethod.ActiveDirectoryInteractive. (file: Modules/ScriptFactoryAccess.cs)
  • Updated object-explorer connection string construction to call the new ApplyAuthentication flow and preserve Encrypt/TrustServerCertificate when available, while avoiding accidental IntegratedSecurity=True for Entra interactive/MFA flows. (file: Modules/ScriptFactoryAccess.cs)
  • Broadened token/keyword detection to recognize both “Microsoft Entra” and common Azure AD/Active Directory authentication keywords so various connection string/property patterns are handled.

Testing

  • Ran git diff --check to verify no whitespace/patch errors, which completed with no issues.
  • Attempted dotnet build AxialSqlTools/AxialSqlTools.sln but the dotnet CLI is not available in the environment so a full build could not be executed.

Codex Task

@alex-bochkov
alex-bochkov merged commit 7a47bec into main May 17, 2026
@alex-bochkov
alex-bochkov deleted the codex/validate-data-transfer-window-support branch May 17, 2026 15:22
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.

1 participant