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

ServerHeartbeatStartedEvent is missing the `awaited` field

    • 3
    • Not Needed
    • Hide

      Create a copy of the Kickoff Template with the issue key (NODE-XXX) in the filename and share a link to the new doc via this field.

      Show
      Create a copy of the Kickoff Template with the issue key (NODE-XXX) in the filename and share a link to the new doc via this field.
    • Not Needed
    • Hide

      1. What would you like to communicate to the user about this feature?
      2. Would you like the user to see examples of the syntax and/or executable code and its output?
      3. Which versions of the driver/connector does this apply to?

      Show
      1. What would you like to communicate to the user about this feature? 2. Would you like the user to see examples of the syntax and/or executable code and its output? 3. Which versions of the driver/connector does this apply to?

      What problem are you facing?

      When reviewing https://github.com/mongodb/specifications/pull/1452, I noticed that the prose tests depend on a property on `ServerHeartbeatStartedEvent`s that does not exist in the Node driver.  After digging though the specs repo, it appears that this field has been a part of the `ServerHeartbeatStartedEvent` since the streamable monitoring protocol was added in 2020 but it was never added in Node.

      The property in question is the `awaited` property of `ServerHeartbeatStartedEvents` (defined here).  

       

      I linked NODE-5197 because it will be necessary for the prose tests we will be adding but we could tackle them together.

      There shouldn't be downstream impact because this is a new feature.

      What driver and relevant dependency versions are you using?

      All drivers since the streaming protocol was added.

      Steps to reproduce?

      n/a

            Assignee:
            durran.jordan@mongodb.com Durran Jordan
            Reporter:
            bailey.pearson@mongodb.com Bailey Pearson
            Bailey Pearson
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: