From efb5af0c476253b29ecd1fcaef64b6a33a8a7afa Mon Sep 17 00:00:00 2001
From: Cheena Malhotra <13396919+cheenamalhotra@users.noreply.github.com>
Date: Tue, 18 Jul 2023 14:59:22 -0700
Subject: [PATCH] Send user as empty string (default value) (#2151)

---
 .../Connection/ConnectionService.cs                           | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/Microsoft.SqlTools.ServiceLayer/Connection/ConnectionService.cs b/src/Microsoft.SqlTools.ServiceLayer/Connection/ConnectionService.cs
index 40360d4e..fa82e612 100644
--- a/src/Microsoft.SqlTools.ServiceLayer/Connection/ConnectionService.cs
+++ b/src/Microsoft.SqlTools.ServiceLayer/Connection/ConnectionService.cs
@@ -1732,7 +1732,9 @@ namespace Microsoft.SqlTools.ServiceLayer.Connection
                 ServerName = defaultBuilder.DataSource != builder.DataSource ? builder.DataSource : null,
                 TrustServerCertificate = defaultBuilder.TrustServerCertificate != builder.TrustServerCertificate ? builder.TrustServerCertificate : false,
                 TypeSystemVersion = defaultBuilder.TypeSystemVersion != builder.TypeSystemVersion ? builder.TypeSystemVersion : null,
-                UserName = defaultBuilder.UserID != builder.UserID ? builder.UserID : null,
+                // !!! ALERT - DO NOT CHANGE USER !!!
+                // SSMS 19 treats "user" as mandatory, always set it to value from connection string builder, even if it's an empty string.
+                UserName = builder.UserID,
                 WorkstationId = defaultBuilder.WorkstationID != builder.WorkstationID ? builder.WorkstationID : null
             };