Only fail if no signature present at all

grafana · Feb 3, 2023 · 5f476db · 5f476db
1 parent 8197ae7
commit 5f476db
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/service_provider.go b/service_provider.go
@@ -1547,7 +1547,7 @@ func (sp *ServiceProvider) ValidateLogoutResponseRedirect(query url.Values) erro
 	}
 
 	if err := sp.validateSignature(doc.Root()); err != nil {
-		if err != errSignatureElementNotPresent && !hasValidSignature {
+		if err != errSignatureElementNotPresent || !hasValidSignature {
 			retErr.PrivateErr = err
 			return retErr
 		}