Change return type of from_sdk_creds, add deps: to Cargo.toml

Author: JamieTsai1024

Cargo.toml

@@ -41,7 +41,7 @@ dns-resolver = ["dep:hickory-resolver", "dep:hickory-proto"]
 cert-key-password = ["dep:pem", "dep:pkcs8"]
 
 # Enable support for MONGODB-AWS authentication.
-aws-auth = ["dep:reqwest", "aws-config", "aws-types", "aws-credential-types"]
+aws-auth = ["dep:reqwest", "dep:aws-config", "dep:aws-types", "dep:aws-credential-types"]
 
 # Enable support for on-demand Azure KMS credentials.
 azure-kms = ["dep:reqwest"]

src/client/auth/aws.rs

@@ -119,12 +119,7 @@ async fn authenticate_stream_inner(
     let creds = get_aws_credentials(credential).await.map_err(|e| {
         Error::authentication_error(MECH_NAME, &format!("failed to get creds: {e}"))
     })?;
-    let aws_credential = AwsCredential::from_sdk_creds(
-        creds.access_key_id().to_string(),
-        creds.secret_access_key().to_string(),
-        creds.session_token().map(|s| s.to_string()),
-        None,
-    );
+    let aws_credential = AwsCredential::from_sdk_creds(creds);
 
     let date = Utc::now();
 
@@ -301,17 +296,12 @@ impl AwsCredential {
     }
 
     // Creates AwsCredential from keys.
-    fn from_sdk_creds(
-        access_key_id: String,
-        secret_access_key: String,
-        session_token: Option<String>,
-        expiration: Option<crate::bson::DateTime>,
-    ) -> Self {
+    fn from_sdk_creds(creds: Credentials) -> Self {
         Self {
-            access_key_id,
-            secret_access_key,
-            session_token,
-            expiration,
+            access_key_id: creds.access_key_id().to_string(),
+            secret_access_key: creds.secret_access_key().to_string(),
+            session_token: creds.session_token().map(|s| s.to_string()),
+            expiration: None,
         }
     }