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

Fix BDC deployment removing all security settings from config (#14756)

Browse Source
This commit is contained in:
Charles Gagnon
2021-03-16 15:44:21 -07:00
committed by GitHub
parent 784d76b886
commit 89e5ee3c80
2 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
extensions/resource-deployment/src/services/azdataService.ts
View File

@@ -61,7 +61,7 @@ export class AzdataService implements IAzdataService {
const env: NodeJS.ProcessEnv = {};
// azdata requires this environment variables to be set
env['ACCEPT_EULA'] = 'yes';
await this.platformService.runCommand(`azdata bdc config init --source ${profileName} --target ${profileName} --force`, { workingDirectory: this.platformService.storagePath(), additionalEnvironmentVariables: env });
await this.platformService.runCommand(`azdata bdc config init --source ${profileName} --path ${profileName} --force`, { workingDirectory: this.platformService.storagePath(), additionalEnvironmentVariables: env });
const configObjects = await Promise.all([
this.getJsonObjectFromFile(path.join(this.platformService.storagePath(), profileName, 'bdc.json')),
this.getJsonObjectFromFile(path.join(this.platformService.storagePath(), profileName, 'control.json'))

6
extensions/resource-deployment/src/services/bigDataClusterDeploymentProfile.ts
View File

@@ -270,10 +270,10 @@ export class BigDataClusterDeploymentProfile {
}
public setAuthenticationMode(mode: string): void {
// If basic authentication is picked, the security section must be removed
// If basic authentication is picked, the activeDirectory security section must be removed
// otherwise azdata will throw validation error
if (mode === AuthenticationMode.Basic && 'security' in this._controlConfig) {
delete this._controlConfig.security;
if (mode === AuthenticationMode.Basic && 'security' in this._controlConfig && 'activeDirectory' in this._controlConfig.security) {
delete this._controlConfig.security.activeDirectory;
}
}