fix: source is valid and fully-qualified url (#524) (#525)

gabo1208 · embano1 · web-flow · commit 053feda3ff03 · 2023-06-26T19:07:47.000+02:00
The source property of the adapter was set using the URL host field which is incomplete and confusing for users migrating from the VEBA project to Sources. This fix changes the source semantic to be a valid and fully-qualifed URL of the configured vCenter event source. Closes: #523 Signed-off-by: Michael Gasch <15986659+embano1@users.noreply.github.com> Co-authored-by: Michael Gasch <15986659+embano1@users.noreply.github.com>
diff --git a/pkg/vsphere/adapter.go b/pkg/vsphere/adapter.go
@@ -73,7 +73,7 @@ func NewAdapter(ctx context.Context, processed adapter.EnvConfigAccessor, ceClie
 		logger.Fatalf("unable to create vSphere client: %v", err)
 	}
 
-	source := vClient.URL().Host
+	source := vClient.URL().String()
 	if source == "" {
 		logger.Fatal("unable to determine vSphere client source: empty host")
 	}

Original file line number	Diff line number	Diff line change
`@@ -73,7 +73,7 @@ func NewAdapter(ctx context.Context, processed adapter.EnvConfigAccessor, ceClie`
`73`	`73`	`logger.Fatalf("unable to create vSphere client: %v", err)`
`74`	`74`	`}`
`75`	`75`
`76`		`- source := vClient.URL().Host`
	`76`	`+ source := vClient.URL().String()`
`77`	`77`	`if source == "" {`
`78`	`78`	`logger.Fatal("unable to determine vSphere client source: empty host")`
`79`	`79`	`}`