Skip to content

Conversation

@bhaan
Copy link
Contributor

@bhaan bhaan commented Jul 22, 2016

In the case that checkTargetTooHigh returns true, we're exiting the logon handling before calling OnLogon. Even though a logon immediately starts in the resend state, we should still callback to the application that the logon completed.

@bhaan bhaan added the Bug Behavior not matching expected label Jul 22, 2016
@cbusbey cbusbey merged commit 77ccec7 into quickfixgo:master Jul 22, 2016
@cbusbey cbusbey added this to the v0.4.0 milestone Jul 22, 2016
@bhaan bhaan deleted the on-logon-bug branch July 25, 2016 13:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Behavior not matching expected

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants