Merge pull request #362 from cbusbey/reset_logging

cbusbey · web-flow · commit bbe42d9f6220 · 2019-02-07T07:47:57.000-06:00
fixes some confusing logging on reset flag handling
diff --git a/session.go b/session.go
@@ -415,8 +415,8 @@ func (s *session) handleLogon(msg *Message) error {
 	var resetSeqNumFlag FIXBoolean
 	if err := msg.Body.GetField(tagResetSeqNumFlag, &resetSeqNumFlag); err == nil {
 		if resetSeqNumFlag {
-			s.log.OnEvent("Logon contains ResetSeqNumFlag=Y, resetting sequence numbers to 1")
 			if !s.sentReset {
+				s.log.OnEvent("Logon contains ResetSeqNumFlag=Y, resetting sequence numbers to 1")
 				resetStore = true
 			}
 		}

Original file line number	Diff line number	Diff line change
`@@ -415,8 +415,8 @@ func (s session) handleLogon(msg Message) error {`
`415`	`415`	`var resetSeqNumFlag FIXBoolean`
`416`	`416`	`if err := msg.Body.GetField(tagResetSeqNumFlag, &resetSeqNumFlag); err == nil {`
`417`	`417`	`if resetSeqNumFlag {`
`418`		`- s.log.OnEvent("Logon contains ResetSeqNumFlag=Y, resetting sequence numbers to 1")`
`419`	`418`	`if !s.sentReset {`
	`419`	`+ s.log.OnEvent("Logon contains ResetSeqNumFlag=Y, resetting sequence numbers to 1")`
`420`	`420`	`resetStore = true`
`421`	`421`	`}`
`422`	`422`	`}`