Skip to content

Commit

Permalink
Removing maskInputs for general settings (#21)
Browse files Browse the repository at this point in the history
* removing maskInputs for general settings
  • Loading branch information
aksm-ms authored Dec 2, 2020
1 parent cc5dcad commit 1808c4f
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 5 deletions.
2 changes: 1 addition & 1 deletion lib/Utils.js
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ class Utils {
static validateSettings(customSettings, maskInputs) {
try {
var customParsedSettings = JSON.parse(customSettings);
if (maskInputs !== "false") {
if (maskInputs !== undefined && maskInputs !== "false") {
Utils.maskValues(customParsedSettings);
}
return customParsedSettings;
Expand Down
2 changes: 1 addition & 1 deletion lib/main.js
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@ function main() {
yield appServiceUtility.updateConnectionStrings(customConnectionStrings);
}
if (ConfigurationSettings) {
let customConfigurationSettings = Utils_1.Utils.validateSettings(ConfigurationSettings, maskInputs);
let customConfigurationSettings = Utils_1.Utils.validateSettings(ConfigurationSettings);
yield appServiceUtility.updateConfigurationSettings(customConfigurationSettings);
}
applicationURL = yield appServiceUtility.getApplicationURL();
Expand Down
4 changes: 2 additions & 2 deletions src/Utils.ts
Original file line number Diff line number Diff line change
@@ -1,10 +1,10 @@
import * as core from '@actions/core';

export class Utils {
static validateSettings(customSettings: string, maskInputs: string) {
static validateSettings(customSettings: string, maskInputs?: string) {
try {
var customParsedSettings = JSON.parse(customSettings);
if (maskInputs !== "false") {
if (maskInputs !== undefined && maskInputs !== "false") {
Utils.maskValues(customParsedSettings);
}
return customParsedSettings;
Expand Down
2 changes: 1 addition & 1 deletion src/main.ts
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ async function main() {
}

if(ConfigurationSettings) {
let customConfigurationSettings = Utils.validateSettings(ConfigurationSettings, maskInputs);
let customConfigurationSettings = Utils.validateSettings(ConfigurationSettings);
await appServiceUtility.updateConfigurationSettings(customConfigurationSettings);
}

Expand Down

0 comments on commit 1808c4f

Please sign in to comment.