ckaczor/azuredatastudio
1
0
Fork 0
mirror of https://github.com/ckaczor/azuredatastudio.git synced 2026-02-04 01:25:38 -05:00
Code Issues Packages Projects Releases Wiki Activity

remove password for connection string non sql auth types (#19484)

Browse Source 
* remove password for connection string non sql auth types

* add comment

* fix test
This commit is contained in:
Vasu Bhog
2022-05-23 16:01:55 -07:00
committed by GitHub
parent c9c670cc86
commit e1fc230273
2 changed files with 7 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
extensions/sql-bindings/src/common/azureFunctionsUtils.ts
View File

@@ -578,6 +578,12 @@ export async function promptConnectionStringPasswordAndUpdateConnectionString(co
// or user chooses to not include password (or if user cancels out of include password prompt), or authentication type is not SQL login
connectionString = await vscodeMssqlApi.getConnectionString(connectionDetails, false, false);
if (connectionInfo.authenticationType !== 'SqlLogin') {
// temporarily fix until STS is fix to not include the placeholder: https://github.com/microsoft/sqltoolsservice/issues/1508
// if authentication type is not SQL login, remove password in connection string
connectionString = connectionString.replace(`Password=${constants.passwordPlaceholder};`, '');
}
if (!connectionInfo.password && connectionInfo.authenticationType === 'SqlLogin') {
// if a connection exists but does not have password saved we ask user if they would like to enter it and save it in local.settings.json
userPassword = await vscode.window.showInputBox({

2
extensions/sql-bindings/src/test/common/azureFunctionsUtils.test.ts
View File

@@ -177,7 +177,7 @@ describe('Tests to verify Azure Functions Utils functions', function (): void {
should(quickInputSpy.notCalled).be.true('showInputBox should not have been called');
should(warningSpy.notCalled).be.true('showWarningMessage should not have been called');
// returned connection string should NOT include password
should(getConnectionString).equals(`Server=${connectionInfo.server};Initial Catalog=${connectionInfo.database};User ID=${connectionInfo.user};Password=${constants.passwordPlaceholder};`);
should(getConnectionString).equals(`Server=${connectionInfo.server};Initial Catalog=${connectionInfo.database};User ID=${connectionInfo.user};`);
});
it('Should ask user to enter password and set password to connection string when connection info does not contain password and auth type is SQL', async () => {