Uploaded image for project: 'QuickFIX/J'
  1. QuickFIX/J
  2. QFJ-629

CLONE - Regression: QFJ-419 change forces ReconnectInterval to be in DEFAULT section

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Default
    • Resolution: Fixed
    • Affects Version/s: 1.5.0
    • Fix Version/s: 1.5.1
    • Component/s: None
    • Labels:
      None

      Description

      I'm testing against the SVN trunk and one of my working configs failed with "ReconnectInterval not defined". This config has the ReconnectInterval defined only in the [SESSION] sections, and not in the [DEFAULT]. Simple one-line fix; see below.

      Index: core/src/main/java/quickfix/DefaultSessionFactory.java
      ===================================================================
      — core/src/main/java/quickfix/DefaultSessionFactory.java (revision 950)
      +++ core/src/main/java/quickfix/DefaultSessionFactory.java (working copy)
      @@ -306,7 +306,7 @@
      private int[] getLogonIntervalsInSeconds(SessionSettings settings, SessionID sessionID) throws ConfigError {
      if (settings.isSetting(sessionID, Initiator.SETTING_RECONNECT_INTERVAL)) {
      try

      { - String raw = settings.getString(Initiator.SETTING_RECONNECT_INTERVAL); + String raw = settings.getString(sessionID, Initiator.SETTING_RECONNECT_INTERVAL); int[] ret = SessionSettings.parseSettingReconnectInterval(raw); if (ret != null) return ret; }

      catch (Throwable e) {

        Attachments

          Activity

            People

            • Assignee:
              ed1978 Eric Deshayes
              Reporter:
              kennysy Kenny Sy
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: