From c41057bfff21b996b75d9348c29e6177542de756 Mon Sep 17 00:00:00 2001 From: Jason Yellick Date: Thu, 7 Sep 2017 13:45:47 -0400 Subject: [PATCH] [FAB-6076] Fix typo in implicit policy log msg There is a log message which was recently changed which logs that: "Only %d policies were satisfied, but needed %d" But the value of the first parameter is being set to the number of remaining policies to be satisfied, not the number that were satisfied. This CR fixes it. Change-Id: Ife82fbe1792fd5dc6a672158cd12dfb02fb9695a Signed-off-by: Jason Yellick --- common/policies/implicitmeta.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/common/policies/implicitmeta.go b/common/policies/implicitmeta.go index 4de78116e15..045b983f7a9 100644 --- a/common/policies/implicitmeta.go +++ b/common/policies/implicitmeta.go @@ -74,7 +74,7 @@ func (imp *implicitMetaPolicy) Evaluate(signatureSet []*cb.SignedData) error { // This log message may be large and expensive to construct, so worth checking the log level if logger.IsEnabledFor(logging.DEBUG) { var b bytes.Buffer - b.WriteString(fmt.Sprintf("Evaluation Failed: Only %d policies were satisfied, but needed %d of [ ", remaining, imp.threshold)) + b.WriteString(fmt.Sprintf("Evaluation Failed: Only %d policies were satisfied, but needed %d of [ ", imp.threshold-remaining, imp.threshold)) for m := range imp.managers { b.WriteString(m) b.WriteString(".")