Remove force user agent option

Author: jesusfcr

cmd/vulcan-zap/main.go

@@ -48,12 +48,11 @@ type options struct {
 }
 
 type tmplOptions struct {
-	Opts           options
-	URL            string
-	IncludePath    string
-	Dir            string
-	UserAgent      string
-	ForceUserAgent bool // DOUBT: This is the only way to ensure all the requests have the same User-Agent. Do we want to force it?
+	Opts        options
+	URL         string
+	IncludePath string
+	Dir         string
+	UserAgent   string
 }
 
 const configTemplate = `
@@ -63,19 +62,6 @@ env:
     urls: [ "{{ .URL }}" ]
     includePaths: [ "{{ .IncludePath }}" ]
 jobs:
-{{ if .ForceUserAgent }}
-- type: script
-  parameters:
-    name: set-user-agent
-    type: httpsender
-    action: add
-    engine: "ECMAScript : Graal.js"
-    inline: |-
-      function sendingRequest(msg, initiator, helper) {
-        msg.getRequestHeader().setHeader("User-Agent", "{{ .UserAgent }}");
-      }
-      function responseReceived(msg, initiator, helper) {}
-{{ end }}
 {{ if .Opts.OpenapiUrl }}
 - type: openapi
   parameters:
@@ -188,7 +174,7 @@ var portPool *pool = createPortPool(13000, 5)
 
 func main() {
 	run := func(ctx context.Context, target, assetType, optJSON string, state checkstate.State) (err error) {
-		logger := check.NewCheckLog(checkName)
+		logger := check.NewCheckLogFromContext(ctx, checkName)
 		var opt options
 		if optJSON != "" {
 			if err = json.Unmarshal([]byte(optJSON), &opt); err != nil {
@@ -229,12 +215,11 @@ func main() {
 		}
 		sb := new(strings.Builder)
 		err = tmpl.Execute(sb, tmplOptions{
-			Opts:           opt,
-			URL:            targetURL.String(),
-			Dir:            tempDir,
-			IncludePath:    includePathRegex,
-			UserAgent:      userAgent,
-			ForceUserAgent: false,
+			Opts:        opt,
+			URL:         targetURL.String(),
+			Dir:         tempDir,
+			IncludePath: includePathRegex,
+			UserAgent:   userAgent,
 		})
 		if err != nil {
 			return fmt.Errorf("unable to execute template: %w", err)