Failure to parse hello response from server

[yourlogarithm]

Versions/Environment

1. What version of Rust are you using?

   • rustc 1.77.0-nightly (5d3d3479d 2024-01-23)

2. What operating system are you using?

   • System: Arch Linux
   • Kernel: Linux 6.7.3-arch1-2
   • Architecture: x86-64
   • Hardware Vendor: Dell Inc.
   • Hardware Model: Latitude 5531

3. What versions of the driver and its dependencies are you using?

   • registry+https://github.com/rust-lang/crates.io-index#mongodb@2.8.0
   • registry+https://github.com/rust-lang/crates.io-index#bson@2.9.0

4. What version of MongoDB are you using? (Check with the MongoDB shell using db.version())

   • 4.4.12

5. What is your MongoDB topology (standalone, replica set, sharded cluster, serverless)?

   • replica set

Describe the bug

connectionId expected to be i64 but is Double

To Reproduce

1. Modify this function for a more verbose error:

pub(crate) fn into_hello_reply(self) -> Result<HelloReply> {
  match self.body::<CommandResponse<HelloCommandResponse>>() {
      Ok(response) if response.is_success() => {
          let server_address = self.source_address().clone();
          let cluster_time = response.cluster_time().cloned();
          Ok(HelloReply {
              server_address,
              command_response: response.body,
              cluster_time,
              raw_command_response: self.into_raw_document_buf(),
          })
      },
      Ok(_) => {
          // Handle the case where the response is Ok but does not meet is_success condition
          Err(ErrorKind::InvalidResponse {
              message: String::from("The command response was not successful.")
          }.into())
      },
      Err(err0) => match self.body::<CommandResponse<CommandErrorBody>>() {
          Ok(command_error_body) => Err(Error::new(
              ErrorKind::Command(command_error_body.body.command_error),
              command_error_body.body.error_labels,
          )),
          Err(err1) => Err(ErrorKind::InvalidResponse {
              message: format!("invalid server response: {err0:?}\nbody: {:?}", self.raw.to_document())
          }
          .into()),
      },
  }
}

2. Use this connection function

async fn get_db(var: &str, db_name: &str) -> Result<mongodb::Database, Box<dyn Error>> {
  let mut options = mongodb::options::ClientOptions::parse(env::var(var)?).await?;
  options.app_name = Some("meta-validator".to_string());
  // println!("{:?}", options);
  let client = mongodb::Client::with_options(options)?;
  let databases = client.list_database_names(None, None).await?;
  debug!("Connected to {}: {:?}", db_name, databases);
  Ok(client.database(db_name))
}

3. Get the error:

Error { kind: InvalidResponse { message: \"invalid type: floating point `1162786981.0`, expected i64\" }, labels: {}, wire_version: None, source: None }
body: Ok(Document({ ..., \"connectionId\": Double(1162786981.0), \"minWireVersion\": Int32(0), ... }))

[abr-egn]

Thank you for the detailed bug report! I'm looking into this now.

[abr-egn]

I've reproduced this locally; we'll have a fix out shortly and will cut a patch release to include it.

[abr-egn]

We just released version 2.8.1 of the driver which fixes this issue. Again, thanks for the report!