Merge pull request robbertkl#59 from bephinix/fix/invalid-debug-msg-rules-insert

robbertkl · web-flow · commit 9c683bf96e1a · 2020-05-27T22:20:41.000+02:00
Fix Debug Log Messages to use correct Parameter for Insert
diff --git a/firewall.go b/firewall.go
@@ -191,7 +191,7 @@ func (fw *Firewall) EnsureRules(rules *Ruleset) error {
 				return err
 			}
 			if fw.debug {
-				log.Println("rule added: -t", string(rule.tc.table), "-A", string(rule.tc.chain), len(fw.activeRules[rule.tc])+1, strings.Join(rule.spec, " "))
+				log.Println("rule added: -t", string(rule.tc.table), "-I", string(rule.tc.chain), len(fw.activeRules[rule.tc])+1, strings.Join(rule.spec, " "))
 			}
 		}
 		fw.activateRule(rule)
@@ -214,7 +214,7 @@ func (fw *Firewall) EnsureRules(rules *Ruleset) error {
 				return err
 			}
 			if fw.debug {
-				log.Println("rule added: -t", string(rule.tc.table), "-A", string(rule.tc.chain), 1, strings.Join(rule.spec, " "))
+				log.Println("rule added: -t", string(rule.tc.table), "-I", string(rule.tc.chain), 1, strings.Join(rule.spec, " "))
 			}
 		}
 		fw.activateRule(rule)

Original file line number	Diff line number	Diff line change
`@@ -191,7 +191,7 @@ func (fw Firewall) EnsureRules(rules Ruleset) error {`
`191`	`191`	`return err`
`192`	`192`	`}`
`193`	`193`	`if fw.debug {`
`194`		`- log.Println("rule added: -t", string(rule.tc.table), "-A", string(rule.tc.chain), len(fw.activeRules[rule.tc])+1, strings.Join(rule.spec, " "))`
	`194`	`+ log.Println("rule added: -t", string(rule.tc.table), "-I", string(rule.tc.chain), len(fw.activeRules[rule.tc])+1, strings.Join(rule.spec, " "))`
`195`	`195`	`}`
`196`	`196`	`}`
`197`	`197`	`fw.activateRule(rule)`
`@@ -214,7 +214,7 @@ func (fw Firewall) EnsureRules(rules Ruleset) error {`
`214`	`214`	`return err`
`215`	`215`	`}`
`216`	`216`	`if fw.debug {`
`217`		`- log.Println("rule added: -t", string(rule.tc.table), "-A", string(rule.tc.chain), 1, strings.Join(rule.spec, " "))`
	`217`	`+ log.Println("rule added: -t", string(rule.tc.table), "-I", string(rule.tc.chain), 1, strings.Join(rule.spec, " "))`
`218`	`218`	`}`
`219`	`219`	`}`
`220`	`220`	`fw.activateRule(rule)`