Fixed failing prometheus tests and linter errors

jhiemstrawisc · Oct 6, 2023 · 09b9a42 · 09b9a42
1 parent 6a98012
commit 09b9a42
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/cmd/origin_serve.go b/cmd/origin_serve.go
@@ -83,8 +83,8 @@ type (
 	}
 
 	ServerConfig struct {
-		TLSCertificate          string
-		TLSKey                  string
+		TLSCertificate            string
+		TLSKey                    string
 		TLSCACertificateDirectory string
 		TLSCACertificateFile      string
 	}

diff --git a/web_ui/prometheus_test.go b/web_ui/prometheus_test.go
@@ -22,6 +22,7 @@ import (
 	"github.com/pelicanplatform/pelican/config"
 	"github.com/pelicanplatform/pelican/param"
 	"github.com/prometheus/common/route"
+	"github.com/spf13/viper"
 	"github.com/stretchr/testify/assert"
 )
 
@@ -101,7 +102,7 @@ func TestPrometheusProtectionFederationURL(t *testing.T) {
 	}
 	jti := base64.RawURLEncoding.EncodeToString(jti_bytes)
 
-	originUrl := param.Origin_Url.GetString
+	originUrl := param.Origin_Url.GetString()
 	tok, err := jwt.NewBuilder().
 		Claim("scope", "prometheus.read").
 		Claim("wlcg.ver", "1.0").
@@ -193,7 +194,7 @@ func TestPrometheusProtectionOriginHeaderScope(t *testing.T) {
 	}
 	jti := base64.RawURLEncoding.EncodeToString(jti_bytes)
 
-	originUrl := param.Origin_Url.GetString
+	originUrl := param.Origin_Url.GetString()
 	tok, err := jwt.NewBuilder().
 		Claim("scope", "prometheus.read").
 		Claim("wlcg.ver", "1.0").