comma seperator incorrect encoding issue fix (#436)

* comma seperator incorrect encoding issue fix

* modified featureflag conditions

* fixed white space issue at commaseperatorcheck

* modified local variable names to camel case

Author: v-dko

common-npm-packages/webdeployment-common/msdeployutility.ts

@@ -113,13 +113,15 @@ export function getMSDeployCmdArgs(webAppPackage: string, webAppName: string, pr
 function escapeQuotes(additionalArguments: string): string {
     const parsedArgs = parseAdditionalArguments(additionalArguments);
     const separator = ",";
-
+    let commaSeperatedCSEnabled = tl.getPipelineFeature('CommaSeperatedConnectionString');
     const formattedArgs = parsedArgs.map(function (arg) {
         let formattedArg = '';
         let equalsSignEncountered = false;
         for (let i = 0; i < arg.length; i++) {
-            const char = arg.charAt(i);
-            if (char == separator && equalsSignEncountered) {
+            const char = arg.charAt(i); 
+            let quotedStringCheck =  (char == separator && equalsSignEncountered && ((formattedArg.startsWith("'") && formattedArg.endsWith("'")) || (formattedArg.startsWith('"') && formattedArg.endsWith('"'))));     
+            let commaSeperatorCheck = commaSeperatedCSEnabled ? quotedStringCheck : (char == separator && equalsSignEncountered);
+            if (commaSeperatorCheck) {
                 equalsSignEncountered = false;
                 arg = arg.replace(formattedArg, escapeArg(formattedArg));
                 formattedArg = '';
@@ -421,4 +423,4 @@ export interface WebDeployResult {
     isSuccess: boolean;
     errorCode?: string;
     error?: string;
-}
+}