Merge pull request #890 from linzworld/patch-1

fix: Set the correct value for logExplains
casbin · Sep 18, 2021 · 8987b14 · 8987b14
2 parents 9253c81 + 94db050
commit 8987b14
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/enforcer.go b/enforcer.go
@@ -664,10 +664,13 @@ func (e *Enforcer) enforce(matcher string, explains *[]string, rvals ...interfac
 	var logExplains [][]string
 
 	if explains != nil {
-		logExplains = append(logExplains, *explains)
-
+		if len(*explains) > 0 {
+			logExplains = append(logExplains, *explains)
+		}
+
 		if explainIndex != -1 && len(e.model["p"][pType].Policy) > explainIndex {
 			*explains = e.model["p"][pType].Policy[explainIndex]
+			logExplains = append(logExplains, *explains)
 		}
 	}