From 4b28e6886cdba86ac03f5b74c88947c238c80600 Mon Sep 17 00:00:00 2001
From: Nick Meves <nick.meves@greenhouse.io>
Date: Tue, 22 Dec 2020 21:34:15 -0800
Subject: [PATCH] Handle ValidateURL fallback for nil & empty struct cases

---
 providers/keycloak.go | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/providers/keycloak.go b/providers/keycloak.go
index 03d3194c..a1e4f064 100644
--- a/providers/keycloak.go
+++ b/providers/keycloak.go
@@ -64,12 +64,12 @@ func NewKeycloakProvider(p *ProviderData) *KeycloakProvider {
 // email and groups.
 func (p *KeycloakProvider) EnrichSession(ctx context.Context, s *sessions.SessionState) error {
 	// Fallback to ValidateURL if ProfileURL not set for legacy compatibility
-	userinfoURL := p.ValidateURL.String()
-	if p.ProfileURL != nil {
-		userinfoURL = p.ProfileURL.String()
+	profileURL := p.ValidateURL.String()
+	if p.ProfileURL.String() != "" {
+		profileURL = p.ProfileURL.String()
 	}
 
-	json, err := requests.New(userinfoURL).
+	json, err := requests.New(profileURL).
 		WithContext(ctx).
 		SetHeader("Authorization", "Bearer "+s.AccessToken).
 		Do().