Merge pull request #1383 from topcoder-platform/alert-autofix-63

Potential fix for code scanning alert no. 63: Inefficient regular expression

Author: kkartunov

src/apps/profiles/src/lib/helpers.ts

@@ -90,7 +90,7 @@ export function subTrackLabelToHumanName(label: string): string {
 export function isValidURL(urlToValidate: string): boolean {
     const pattern = new RegExp(
         '^(https?:\\/\\/)?' // protocol
-        + '((([a-z\\d]([a-z\\d-]*[a-z\\d])*)\\.)+[a-z]{2,}|' // domain name
+        + '((([a-z\\d](?:[a-z\\d-]*[a-z\\d])?)\\.)+[a-z]{2,}|' // domain name, forbid leading/trailing dash in label
         + '((\\d{1,3}\\.){3}\\d{1,3}))' // OR IP (v4) address
         + '(\\:\\d+)?(\\/[-a-z\\d%_.~+]*)*' // port and path
         + '(\\?[;&a-z\\d%_.~+=-]*)?' // query string