[Feature] Using CodeQL alert suppressions rather than LGTM alert suppressions (#1876)

* [Feature] Using CodeQL alert suppressions rather than LGTM alert suppressions
Fixes #1873

* [Feature] Using CodeQL alert suppressions rather than LGTM alert suppressions
Fixes #1873

* [Feature] Using CodeQL alert suppressions rather than LGTM alert suppressions
Fixes #1873

Co-authored-by: Ezio Li <[email protected]>

Author: benjaminbi

src/cdp-proxy/debuggerEndpointHelper.ts

@@ -122,7 +122,7 @@ export class DebuggerEndpointHelper {
             const requestOptions: https.RequestOptions = {};
 
             if (isSecure && targetAddressIsLoopback) {
-                requestOptions.rejectUnauthorized = false; // lgtm [js/disabling-certificate-validation]
+                requestOptions.rejectUnauthorized = false; // CodeQL [js/disabling-certificate-validation] Debug extension does not need to verify certificate
             }
 
             const request = driver.get(url, requestOptions, response => {

tools/gulp-extras.js

@@ -17,7 +17,7 @@ const through = require("through2");
  * @param {string} message The error message to display
  */
 function logError(pluginName, file, message) {
-    const sourcePath = path.relative(__dirname, file.path).replace("../", ""); // lgtm [js/incomplete-sanitization]
+    const sourcePath = path.relative(__dirname, file.path).replace("../", ""); // CodeQL [js/incomplete-sanitization] Debugging extension has no need to use global replacement in file path string
     log(`[${colors.cyan(pluginName)}] ${colors.red("error")} ${sourcePath}: ${message}`);
 }