Remove discovery.acl principal warning

PR hyperledger#3006 added warnings for principal check failures to assist with troubleshooting. The discovery warning was too much however since even in normal scenarios discovery endorser service checks the peer against the various channel principals. This change reverts to the prior code without the warning. Resolves hyperledger/fabric-gateway#349. Signed-off-by: David Enyeart <[email protected]>
CanisMinor-1037 · Dec 13, 2021 · 0b0c35c · 0b0c35c
1 parent 3552917
commit 0b0c35c
Showing 1 changed file with 1 addition and 5 deletions.
diff --git a/discovery/support/acl/support.go b/discovery/support/acl/support.go
@@ -99,11 +99,7 @@ func (s *DiscoverySupport) SatisfiesPrincipal(channel string, rawIdentity []byte
 		logger.Warnw("failed deserializing identity", "error", err, "identity", protoutil.LogMessageForSerializedIdentity(rawIdentity))
 		return errors.Wrap(err, "failed deserializing identity")
 	}
-	err = identity.SatisfiesPrincipal(principal)
-	if err != nil {
-		logger.Warnw("identity does not satisfy principal", "error", err, "requiredPrincipal", principal, "identity", protoutil.LogMessageForSerializedIdentity(rawIdentity))
-	}
-	return err
+	return identity.SatisfiesPrincipal(principal)
 }
 
 // ChannelPolicyManagerGetter is a support interface