Skip to content

Commit

Permalink
CB-5197 do not check configuration type for creds
Browse files Browse the repository at this point in the history
  • Loading branch information
devnaumov committed Jun 27, 2024
1 parent d3afe0e commit 31311f0
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -321,7 +321,7 @@ export class ConnectionOptionsTabService extends Bootstrap {
const properties = await this.getConnectionAuthModelProperties(tempConfig.authModelId, state.info);

if (this.isCredentialsChanged(properties, state.config.credentials)) {
tempConfig.credentials = this.prepareDynamicProperties(properties, toJS(state.config.credentials), tempConfig.configurationType);
tempConfig.credentials = this.prepareDynamicProperties(properties, toJS(state.config.credentials));
}

if (!tempConfig.saveCredentials) {
Expand Down Expand Up @@ -358,7 +358,7 @@ export class ConnectionOptionsTabService extends Bootstrap {
continue;
}

const supported = propertyInfo.supportedConfigurationTypes?.some(type => type === configurationType);
const supported = configurationType === undefined || propertyInfo.supportedConfigurationTypes?.some(type => type === configurationType);

if (!supported) {
delete result[propertyInfo.id];
Expand Down

0 comments on commit 31311f0

Please sign in to comment.