From 537c78dbb000c3e945c9534d2a6a2d758ef80025 Mon Sep 17 00:00:00 2001 From: Cheena Malhotra <13396919+cheenamalhotra@users.noreply.github.com> Date: Tue, 25 Apr 2023 20:09:40 -0700 Subject: [PATCH] Fix no password user scenario (#2032) --- .../Connection/ConnectionService.cs | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/src/Microsoft.SqlTools.ServiceLayer/Connection/ConnectionService.cs b/src/Microsoft.SqlTools.ServiceLayer/Connection/ConnectionService.cs index 1be64041..abd7eb90 100644 --- a/src/Microsoft.SqlTools.ServiceLayer/Connection/ConnectionService.cs +++ b/src/Microsoft.SqlTools.ServiceLayer/Connection/ConnectionService.cs @@ -1377,10 +1377,9 @@ namespace Microsoft.SqlTools.ServiceLayer.Connection break; case SqlLogin: connectionBuilder.UserID = connectionDetails.UserName; - if (!string.IsNullOrEmpty(connectionDetails.Password)) - { - connectionBuilder.Password = connectionDetails.Password; - } + connectionBuilder.Password = string.IsNullOrEmpty(connectionDetails.Password) + ? string.Empty // Support empty password for accounts without password + : connectionDetails.Password; connectionBuilder.Authentication = SqlAuthenticationMethod.SqlPassword; break; case AzureMFA: