Connect-DbaInstance, change username format only when joined to AD

dataplat · Feb 25, 2025 · 4dde0cc · 4dde0cc
1 parent bb33a85
commit 4dde0cc
Showing 1 changed file with 11 additions and 4 deletions.
diff --git a/public/Connect-DbaInstance.ps1 b/public/Connect-DbaInstance.ps1
@@ -693,11 +693,18 @@ function Connect-DbaInstance {
                     $authType = 'local '
                 }
                 if ($SqlCredential) {
-                    # support both ad\username and username@ad
-                    # username@ad works only for domain joined
-                    # so we stick to what we get and allow ad\username
-                    # as well, that works on local and workgroup
                     $username = ($SqlCredential.UserName).TrimStart("\")
+                    # support both ad\username and username@ad
+                    # username@ad works only for domain joined and workgroup
+                    # nobody remembers why, but username@ad is preferred
+                    # so we switch ad\username to username@ad only doing a raw guess
+                    # when USERDOMAIN -ne COMPUTERNAME, we're probably joined to ad
+                    if ($env:USERDOMAIN -ne $env:COMPUTERNAME) {
+                        if ($username -like "*\*") {
+                            $domain, $login = $username.Split("\")
+                            $username = "$login@$domain"
+                        }
+                    }
                     if ($username -like '*@*' -or $username -like '*\*') {
                         $authType += 'ad'
                     } else {