Restore keepAliveInitialDelay configurability

XMLWordPrintableJSON

    • Type: Improvement
    • Resolution: Fixed
    • Priority: Critical - P2
    • 6.17.0
    • Affects Version/s: None
    • Component/s: None
    • 2
    • Not Needed
    • None
    • Not Needed
    • None
    • None
    • None
    • None
    • None
    • None

      These were removed in 6.0 with NODE-5190, however it's unclear what the value of this removal was.

      Let's keep the defaults that have been set but restore the ability to configure keepAliveInitialDelay (note that keepAlive should still not be toggleable)

      Use Case

      As a... driver user
      I want... to be able to configure keepAliveInitialDelay
      So that... I can choose a setting that works for my application

      User Experience

      • Use is able to set keepAliveInitialDelay to a value of their choice.

      Dependencies

      • N/A

      Risks/Unknowns

      • What could go wrong while implementing this change? (e.g., performance, inadvertent behavioral changes in adjacent functionality, existing tech debt, etc)
      • Is there an opportunity for better cross-driver alignment or testing in this area?
      • Is there an opportunity to improve existing documentation on this subject?

      Acceptance Criteria

      Implementation Requirements

      • Ensure users can configure the keepAliveInitialDelay option via client (connection string and constructor), allow it to be a pass-through with the exception of parseOptions validation.
      • Set default value to 120 seconds.

      Testing Requirements

      • Test the pass-through with valid and invalid values (negative numbers, non-numeric types)

      Documentation Requirements

      • API docs
      • Add manual docs for the new option

      Follow Up Requirements

      • None

            Assignee:
            Durran Jordan
            Reporter:
            Alex Bevilacqua
            Neal Beeken
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

              Created:
              Updated:
              Resolved: