Uploaded image for project: 'Node.js Driver'
  1. Node.js Driver
  2. NODE-4509

Use 1 instead of true for hello value

    • Type: Icon: Improvement Improvement
    • Resolution: Fixed
    • Priority: Icon: Trivial - P5 Trivial - P5
    • 5.0.0
    • Affects Version/s: None
    • Component/s: None
    • Labels:
    • 2
    • Not Needed
    • Not Needed

      We currently send hello: true to MongoDB which works fine, it would be nice to align with other drivers and send hello: 1 / [LEGACY_HELLO]: 1. We discussed doing this in the major to be overly cautious about a change to commands sent from the driver. This should have no impact.

      AC

      • make the change
      • Document change in the migration guide
      • add a regression test

            Assignee:
            durran.jordan@mongodb.com Durran Jordan
            Reporter:
            neal.beeken@mongodb.com Neal Beeken
            Neal Beeken
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: