Only fail if no signature present at all

grafana · Oct 25, 2023 · ef8268d · ef8268d
1 parent 96e0d74
commit ef8268d
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/service_provider.go b/service_provider.go
@@ -1563,7 +1563,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
 		}