Merge pull request #563 from DefangLabs/kevin/deployerURL

Author: KevyVo

tools/testing/deployer/deployer.go

@@ -120,24 +120,23 @@ func (d *CliDeployer) Deploy(ctx context.Context) error {
 	return err
 }
 
-var urlRegex = regexp.MustCompile(`DEPLOYMENT_COMPLETED\s*([^\s]+)`)
+var urlRegex = regexp.MustCompile(`DEPLOYMENT_COMPLETED.+(https://\S+)`)
 var internalURLRegex = regexp.MustCompile(`\.internal:\d+$`)
 
 func findUrlsInOutput(output string) []string {
 	var urls []string
 	match := urlRegex.FindAllStringSubmatch(output, -1)
+
+	// Will never be nil if there is a match
 	for _, m := range match {
-		if m[1] == "" {
-			continue
-		}
+		// Skip internal URLs
 		if internalURLRegex.MatchString(m[1]) {
 			log.Printf("Skipping internal URL %v", m[1])
 			continue
 		}
-		// Check if the URL starts with a scheme (http, https, etc.)
-		if strings.HasPrefix(m[1], "https://") {
-			urls = append(urls, m[1])
-		}
+
+		// Add the URL if there a https URL found
+		urls = append(urls, m[1])
 	}
 	return urls
 }