quickfixgo / quickfix

The Go FIX Protocol Library :rocket:
https://www.quickfixgo.org/
Other
723 stars 282 forks source link

Duplicate SessionID #614

Closed FattyMango closed 2 months ago

FattyMango commented 5 months ago

I'm creating a FIX(4.4) client and I'm trying to implement a retry mechanism that stops the entire initiator and starts it all over again, I'm doing this since the Quickfix auto reconnect doesn't send a ResetOnLogon in the reconnect message.

but when I try to create a new initiator with the same session I get a Duplicate SessionID error, I was able to reproduce the error and it appears to be from this function

func registerSession(s *session) error {
    sessionsLock.Lock()
    defer sessionsLock.Unlock()

    if _, ok := sessions[s.sessionID]; ok { // here
        return errDuplicateSessionID
    }

    sessions[s.sessionID] = s
    return nil
}

it works as expected when I remove the session check.

AlexandrosKyriakakis commented 3 months ago

When you create a session (here) there is no way to remove it from this sessions map so when you create another initiator with same configs under the same thread you got the error Duplicate SessionID

We just need to remove the sessions map from global variables

ackleymi commented 2 months ago

Resolved with #637