Skip to content

New-DbaLogin: add ExternalGroup support and SQL Server 2022 Entra login handling#10225

Merged
potatoqualitee merged 1 commit into
developmentfrom
claude/issue-9586-20260310-0726
Mar 10, 2026
Merged

New-DbaLogin: add ExternalGroup support and SQL Server 2022 Entra login handling#10225
potatoqualitee merged 1 commit into
developmentfrom
claude/issue-9586-20260310-0726

Conversation

@andreasjordan
Copy link
Copy Markdown
Collaborator

Fixes #damagetools_9586

Fixes Copy-DbaLogin and Sync-DbaAvailabilityGroup not syncing Azure Entra logins (ExternalUser/ExternalGroup) on SQL Server 2022.

Two issues were addressed:

  1. ExternalGroup login type was not included in the SMO Create() path or property-setting block, causing it to fall through to FROM WINDOWS T-SQL which fails for Entra group logins.
  2. The T-SQL fallback for FROM EXTERNAL PROVIDER only triggered for Azure SQL DB and Managed Instance, but SQL Server 2022 (version 16+) also supports this syntax for Entra logins.

Fixes #9586

Generated with Claude Code

…in handling

Fixes Copy-DbaLogin and Sync-DbaAvailabilityGroup not syncing Azure Entra
logins (ExternalUser/ExternalGroup) on SQL Server 2022.

Two issues were addressed:
1. ExternalGroup login type was not included in the SMO Create() path or
   property-setting block, causing it to fall through to FROM WINDOWS T-SQL
   which fails for Entra group logins.
2. The T-SQL fallback for FROM EXTERNAL PROVIDER only triggered for Azure SQL
   DB and Managed Instance, but SQL Server 2022 (version 16+) also supports
   this syntax for Entra logins.

(do *Login*)

Co-authored-by: Andreas Jordan <andreasjordan@users.noreply.github.com>
@potatoqualitee potatoqualitee merged commit f3e4022 into development Mar 10, 2026
15 checks passed
@potatoqualitee potatoqualitee deleted the claude/issue-9586-20260310-0726 branch March 10, 2026 16:27
@potatoqualitee
Copy link
Copy Markdown
Member

awesome ty 🙏🏼

potatoqualitee added a commit that referenced this pull request Apr 10, 2026
… of #10225)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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.

Sync-DbaAvailabilityGroup not syncing Azure Entra logins

2 participants