Skip to content

Commit

Permalink
[FAB-6076] Fix typo in implicit policy log msg
Browse files Browse the repository at this point in the history
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 <jyellick@us.ibm.com>
  • Loading branch information
Jason Yellick committed Sep 7, 2017
1 parent a657db2 commit c41057b
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion common/policies/implicitmeta.go
Original file line number Diff line number Diff line change
Expand Up @@ -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(".")
Expand Down

0 comments on commit c41057b

Please sign in to comment.