Skip to content

HCK-13038: adjust Azure MFA auth for RE#124

Merged
taras-dubyk merged 3 commits intodevelopfrom
fix/HCK-13038
Oct 14, 2025
Merged

HCK-13038: adjust Azure MFA auth for RE#124
taras-dubyk merged 3 commits intodevelopfrom
fix/HCK-13038

Conversation

@taras-dubyk
Copy link
Contributor

@taras-dubyk taras-dubyk commented Oct 13, 2025

Sub-taskHCK-13038 [Inotive][SQLServer] Enable/fix connection with MS Entra ID to leverage connection to Microsoft Fabric in that plugin (similar to Avro)

Content

This PR adjusts the Azure MFA authentication flow for reverse-engineering.

Changes:

  • Removed @azure/msal-node package as it is no longer used.
  • Renamed the authentication option "Azure Active Directory" to "Azure Entra ID".
  • Removed "Tenant ID" and "User name" parameters for "Azure Entra ID", as they are not used.
  • Adjusted the logic for parsing the database name from the connection string.

@taras-dubyk taras-dubyk self-assigned this Oct 13, 2025
@sonarqubecloud
Copy link

@chulanovskyi-bs chulanovskyi-bs enabled auto-merge (squash) October 14, 2025 10:43
@taras-dubyk taras-dubyk disabled auto-merge October 14, 2025 11:31
@taras-dubyk taras-dubyk merged commit cbc9592 into develop Oct 14, 2025
8 of 9 checks passed
@taras-dubyk taras-dubyk deleted the fix/HCK-13038 branch October 14, 2025 11:31
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