Adds a new task to override virtual network names

also fixes a couple bugs

Author: aarondandy

build.ps1

@@ -0,0 +1,11 @@
+Push-Location .\cscfg-transform
+npm install
+tsc
+Pop-Location
+
+Push-Location .\cscfg-vnetsite
+npm install
+tsc
+Pop-Location
+
+tfx extension create --manifest-globs vss-extension.json

SampleServiceConfiguration.cscfg renamed to cscfg-transform/SampleServiceConfiguration.cscfg

@@ -51,7 +51,7 @@ async function run() {
 
     if (fileMutated) {
       const newContents = new XMLSerializer().serializeToString(doc);
-      fsPromises.writeFile(targetFilePath, newContents);
+      await fsPromises.writeFile(targetFilePath, newContents);
     }
     else {
       tl.warning("No variable updates were applied to the file.");

cscfg-transform/images/icon.png renamed to cscfg-transform/icon.png

@@ -4,8 +4,7 @@
   "description": "Transforms Azure Cloud Service cscfg files",
   "main": "index.js",
   "scripts": {
-    "test": "tsc && mocha ./tests/_suite.js",
-    "package": "tsc && tfx extension create --manifest-globs cscfg-transform-extension.json"
+    "test": "tsc && mocha ./tests/_suite.js"
   },
   "keywords": [
     "cscfg"

cscfg-transform/index.ts

@@ -16,7 +16,7 @@
     "Minor": 2,
     "Patch": 3
   },
-  "instanceNameFormat": "Transform cscfg $(targetFilePath)",
+  "instanceNameFormat": "Transform CS Config Variables $(targetFilePath)",
   "inputs": [
     {
       "name": "targetFilePath",

cscfg-transform/package.json

@@ -0,0 +1,22 @@
+<?xml version="1.0" encoding="utf-8"?>
+<!--
+  **********************************************************************************************
+
+  This file was generated by a tool from the project file: ServiceConfiguration.cscfg
+
+  Changes to this file may cause incorrect behavior and will be lost if the file is regenerated.
+
+  **********************************************************************************************
+-->
+<ServiceConfiguration serviceName="ServiceName" xmlns="http://schemas.microsoft.com/ServiceHosting/2008/10/ServiceConfiguration" osFamily="6" osVersion="*" schemaVersion="2015-04.2.6">
+  <Role name="RoleName">
+    <Instances count="1"/>
+    <ConfigurationSettings>
+      <Setting name="foo.blank" value=""/>
+      <Setting name="foo.bar" value="bar"/>
+    </ConfigurationSettings>
+  </Role>
+  <NetworkConfiguration>
+    <VirtualNetworkSite name=""/>
+  </NetworkConfiguration>
+</ServiceConfiguration>

cscfg-transform/task.json

@@ -0,0 +1,64 @@
+import tl = require('azure-pipelines-task-lib/task');
+import { promises as fsPromises } from 'fs';
+import { DOMParser, XMLSerializer } from 'xmldom';
+
+async function run() {
+  try {
+    const targetFilePath: string | undefined = tl.getInput('targetFilePath', true);
+    if (targetFilePath === '' || targetFilePath == null) {
+      tl.setResult(tl.TaskResult.Failed, 'A file path to transform is required');
+      return;
+    }
+
+    const overrideName: string | undefined = tl.getInput('overrideNetworkSiteName', true);
+    if (overrideName == null) {
+      tl.setResult(tl.TaskResult.Failed, 'An override name is required');
+      return;
+    }
+
+    let fileMutated = false;
+
+    const originalContents = await fsPromises.readFile(targetFilePath, 'utf8');
+    const parser = new DOMParser();
+    
+    let doc = parser.parseFromString(originalContents, 'text/xml');
+    if (doc.documentElement.nodeName !== 'ServiceConfiguration') {
+      tl.setResult(tl.TaskResult.Failed, 'Config file missing root ServiceConfiguration element');
+      return;
+    }
+
+    let networkNode = doc.documentElement.getElementsByTagName('NetworkConfiguration')[0];
+    if (!networkNode) {
+      networkNode = doc.createElement('NetworkConfiguration');
+      doc.documentElement.appendChild(networkNode);
+      fileMutated = true;
+    }
+
+    let virtualNetworkSite = networkNode.getElementsByTagName('VirtualNetworkSite')[0];
+    if (!virtualNetworkSite) {
+      virtualNetworkSite = doc.createElement('VirtualNetworkSite');
+      networkNode.appendChild(virtualNetworkSite);
+      fileMutated = true;
+    }
+
+    const originalName = virtualNetworkSite.getAttribute('name');
+    if (originalName !== overrideName) {
+      virtualNetworkSite.setAttribute('name', overrideName);
+      fileMutated = true;
+    }
+
+    if (fileMutated) {
+      const newContents = new XMLSerializer().serializeToString(doc);
+      await fsPromises.writeFile(targetFilePath, newContents);
+    }
+    else {
+      tl.warning("No updates were applied to the file.");
+    }
+
+  }
+  catch (err) {
+    tl.setResult(tl.TaskResult.Failed, err.message);
+  }
+}
+
+run();