Fix/Suppress CodeQL items (#652)

* Fix/Suppress CodeQL items

* Fix linting issue

Author: andystaples

samples-js/functions/backupSiteContent.js

@@ -20,7 +20,7 @@ df.app.orchestration("backupSiteContent", function* (context) {
     const tasks = [];
     for (const file of files) {
         const input = {
-            backupPath: path.relative(rootDirAbs, file).replace("\\", "/"),
+            backupPath: path.relative(rootDirAbs, file).replace(/\\/g, "/"),
             filePath: file,
         };
         tasks.push(context.df.callActivity(copyFileToBlobActivityName, input));

samples-ts/functions/backupSiteContent.ts

@@ -29,7 +29,7 @@ const backupSiteContentOrchestration: OrchestrationHandler = function* (
     const tasks: Task[] = [];
     for (const file of files) {
         const input = {
-            backupPath: path.relative(rootDirAbs, file).replace("\\", "/"),
+            backupPath: path.relative(rootDirAbs, file).replace(/\\/g, "/"),
             filePath: file,
         };
         tasks.push(context.df.callActivity(copyFileToBlobActivityName, input));

src/util/GuidManager.ts

@@ -18,7 +18,9 @@ export class GuidManager {
         Utils.throwIfEmpty(namespaceValue, "namespaceValue");
         Utils.throwIfEmpty(name, "name");
 
-        const hash = crypto.createHash("sha1");
+        /* More info: The information being hashed in this method is not sensitive, the goal is to generate a deterministic Guid.
+        We cannot update to SHA2-based algorithms without breaking customers' inflight orchestrations */
+        const hash = crypto.createHash("sha1"); // CodeQL [SM04514] SHA1 is not used for cryptographic purposes here
         hash.update(name);
         const bytes: number[] = Array.prototype.slice.call(hash.digest(), 0, 16);