fix(libdd-traceutils): Update cloud environment detection logic for Serverless [SVLS-8799]

libdd-trace-utils/src/config_utils.rs

@@ -3,33 +3,71 @@
 
 use crate::trace_utils;
 use std::env;
+use tracing::{debug, error};
 
 pub const PROD_INTAKE_SUBDOMAIN: &str = "trace.agent";
 
 const TRACE_INTAKE_ROUTE: &str = "/api/v0.2/traces";
 const TRACE_STATS_INTAKE_ROUTE: &str = "/api/v0.2/stats";
 
 pub fn read_cloud_env() -> Option<(String, trace_utils::EnvironmentType)> {
-    if let Ok(res) = env::var("AWS_LAMBDA_FUNCTION_NAME") {
-        return Some((res, trace_utils::EnvironmentType::LambdaFunction));
+    let mut detected: Vec<(String, trace_utils::EnvironmentType)> = Vec::new();
+
+    if env::var("AWS_LAMBDA_INITIALIZATION_TYPE").is_ok() {
+        match env::var("AWS_LAMBDA_FUNCTION_NAME") {
+            Ok(name) => detected.push((name, trace_utils::EnvironmentType::LambdaFunction)),
+            Err(_) => {
+                error!("AWS Lambda environment detected but AWS_LAMBDA_FUNCTION_NAME is not set");
+            }
+        }
     }
-    if let Ok(res) = env::var("K_SERVICE") {
-        // Set by Google Cloud Functions for newer runtimes
-        return Some((res, trace_utils::EnvironmentType::CloudFunction));
+
+    if env::var("FUNCTIONS_EXTENSION_VERSION").is_ok()
+        && env::var("FUNCTIONS_WORKER_RUNTIME").is_ok()
+    {
+        match env::var("WEBSITE_SITE_NAME") {
+            Ok(name) => detected.push((name, trace_utils::EnvironmentType::AzureFunction)),
+            Err(_) => {
+                error!("Azure Functions environment detected but WEBSITE_SITE_NAME is not set");
+            }
+        }
     }
-    if let Ok(res) = env::var("FUNCTION_NAME") {
+
+    if let (Ok(name), Ok(_)) = (env::var("K_SERVICE"), env::var("FUNCTION_TARGET")) {
+        // Set by Google Cloud Functions for newer runtimes
+        detected.push((name, trace_utils::EnvironmentType::CloudFunction));
+    } else if let (Ok(name), Ok(_)) = (env::var("FUNCTION_NAME"), env::var("GCP_PROJECT")) {
         // Set by Google Cloud Functions for older runtimes
-        return Some((res, trace_utils::EnvironmentType::CloudFunction));
+        detected.push((name, trace_utils::EnvironmentType::CloudFunction));
     }
-    if let Ok(res) = env::var("WEBSITE_SITE_NAME") {
-        // Set by Azure Functions
-        return Some((res, trace_utils::EnvironmentType::AzureFunction));
-    }
-    if let Ok(res) = env::var("ASCSVCRT_SPRING__APPLICATION__NAME") {
+
+    if let Ok(name) = env::var("ASCSVCRT_SPRING__APPLICATION__NAME") {
         // Set by Azure Spring Apps
-        return Some((res, trace_utils::EnvironmentType::AzureSpringApp));
+        detected.push((name, trace_utils::EnvironmentType::AzureSpringApp));
+    }
+
+    match detected.len() {
+        0 => {
+            error!("No cloud environment detected");
+            None
+        }
+        1 => {
+            let (ref name, ref env_type) = detected[0];
+            debug!("Cloud environment detected: {env_type:?} ({name})");
+            detected.into_iter().next()
+        }
+        _ => {
+            let env_names: Vec<String> = detected
+                .iter()
+                .map(|(name, env_type)| format!("{env_type:?}({name})"))
+                .collect();
+            error!(
+                "Multiple cloud environments detected: {}",
+                env_names.join(", ")
+            );
+            None
+        }
     }
-    None
 }
 
 pub fn trace_intake_url(site: &str) -> String {
@@ -51,3 +89,195 @@ pub fn trace_stats_url_prefixed(endpoint_prefix: &str) -> String {
 fn construct_trace_intake_url(prefix: &str, route: &str) -> String {
     format!("https://{PROD_INTAKE_SUBDOMAIN}.{prefix}{route}")
 }
+
+#[cfg(test)]
+mod tests {
+    use super::*;
+    use std::sync::Mutex;
+
+    // Mutex to ensure environment variable tests run sequentially
+    static ENV_TEST_LOCK: Mutex<()> = Mutex::new(());
+
+    fn clear_all_env_vars() {
+        unsafe {
+            env::remove_var("AWS_LAMBDA_INITIALIZATION_TYPE");
+            env::remove_var("AWS_LAMBDA_FUNCTION_NAME");
+            env::remove_var("FUNCTIONS_EXTENSION_VERSION");
+            env::remove_var("FUNCTIONS_WORKER_RUNTIME");
+            env::remove_var("WEBSITE_SITE_NAME");
+            env::remove_var("FUNCTION_NAME");
+            env::remove_var("GCP_PROJECT");
+            env::remove_var("K_SERVICE");
+            env::remove_var("FUNCTION_TARGET");
+            env::remove_var("ASCSVCRT_SPRING__APPLICATION__NAME");
+        }
+    }
+
+    #[test]
+    fn test_aws_lambda_detected() {
+        let _lock = ENV_TEST_LOCK.lock().unwrap_or_else(|e| e.into_inner());
+        clear_all_env_vars();
+        unsafe {
+            env::set_var("AWS_LAMBDA_INITIALIZATION_TYPE", "on-demand");
+            env::set_var("AWS_LAMBDA_FUNCTION_NAME", "my-function");
+        }
+        let result = read_cloud_env();
+        assert_eq!(
+            result,
+            Some((
+                "my-function".to_string(),
+                trace_utils::EnvironmentType::LambdaFunction
+            ))
+        );
+    }
+
+    #[test]
+    fn test_aws_lambda_missing_function_name() {
+        let _lock = ENV_TEST_LOCK.lock().unwrap_or_else(|e| e.into_inner());
+        clear_all_env_vars();
+        unsafe { env::set_var("AWS_LAMBDA_INITIALIZATION_TYPE", "on-demand") };
+        let result = read_cloud_env();
+        assert_eq!(result, None);
+    }
+
+    #[test]
+    fn test_aws_lambda_not_detected_without_init_type() {
+        let _lock = ENV_TEST_LOCK.lock().unwrap_or_else(|e| e.into_inner());
+        clear_all_env_vars();
+        unsafe { env::set_var("AWS_LAMBDA_FUNCTION_NAME", "my-function") };
+        let result = read_cloud_env();
+        assert_eq!(result, None);
+    }
+
+    #[test]
+    fn test_azure_function_detected() {
+        let _lock = ENV_TEST_LOCK.lock().unwrap_or_else(|e| e.into_inner());
+        clear_all_env_vars();
+        unsafe {
+            env::set_var("FUNCTIONS_EXTENSION_VERSION", "~4");
+            env::set_var("FUNCTIONS_WORKER_RUNTIME", "java");
+            env::set_var("WEBSITE_SITE_NAME", "my-azure-app");
+        }
+        let result = read_cloud_env();
+        assert_eq!(
+            result,
+            Some((
+                "my-azure-app".to_string(),
+                trace_utils::EnvironmentType::AzureFunction
+            ))
+        );
+    }
+
+    #[test]
+    fn test_azure_function_missing_site_name() {
+        let _lock = ENV_TEST_LOCK.lock().unwrap_or_else(|e| e.into_inner());
+        clear_all_env_vars();
+        unsafe {
+            env::set_var("FUNCTIONS_EXTENSION_VERSION", "~4");
+            env::set_var("FUNCTIONS_WORKER_RUNTIME", "java");
+        }
+        let result = read_cloud_env();
+        assert_eq!(result, None);
+    }
+
+    #[test]
+    fn test_azure_function_not_detected_with_only_one_var() {
+        let _lock = ENV_TEST_LOCK.lock().unwrap_or_else(|e| e.into_inner());
+        clear_all_env_vars();
+        unsafe { env::set_var("FUNCTIONS_EXTENSION_VERSION", "~4") };
+        let result = read_cloud_env();
+        assert_eq!(result, None);
+    }
+
+    #[test]
+    fn test_gcp_1st_gen_detected() {
+        let _lock = ENV_TEST_LOCK.lock().unwrap_or_else(|e| e.into_inner());
+        clear_all_env_vars();
+        unsafe {
+            env::set_var("FUNCTION_NAME", "my-gcp-function");
+            env::set_var("GCP_PROJECT", "my-project");
+        }
+        let result = read_cloud_env();
+        assert_eq!(
+            result,
+            Some((
+                "my-gcp-function".to_string(),
+                trace_utils::EnvironmentType::CloudFunction
+            ))
+        );
+    }
+
+    #[test]
+    fn test_gcp_1st_gen_not_detected_without_gcp_project() {
+        let _lock = ENV_TEST_LOCK.lock().unwrap_or_else(|e| e.into_inner());
+        clear_all_env_vars();
+        unsafe { env::set_var("FUNCTION_NAME", "my-gcp-function") };
+        let result = read_cloud_env();
+        assert_eq!(result, None);
+    }
+
+    #[test]
+    fn test_gcp_2nd_gen_detected() {
+        let _lock = ENV_TEST_LOCK.lock().unwrap_or_else(|e| e.into_inner());
+        clear_all_env_vars();
+        unsafe {
+            env::set_var("K_SERVICE", "my-cloud-run-fn");
+            env::set_var("FUNCTION_TARGET", "myHandler");
+        }
+        let result = read_cloud_env();
+        assert_eq!(
+            result,
+            Some((
+                "my-cloud-run-fn".to_string(),
+                trace_utils::EnvironmentType::CloudFunction
+            ))
+        );
+    }
+
+    #[test]
+    fn test_gcp_2nd_gen_not_detected_without_function_target() {
+        let _lock = ENV_TEST_LOCK.lock().unwrap_or_else(|e| e.into_inner());
+        clear_all_env_vars();
+        unsafe { env::set_var("K_SERVICE", "my-cloud-run-fn") };
+        let result = read_cloud_env();
+        assert_eq!(result, None);
+    }
+
+    #[test]
+    fn test_azure_spring_app_detected() {
+        let _lock = ENV_TEST_LOCK.lock().unwrap_or_else(|e| e.into_inner());
+        clear_all_env_vars();
+        unsafe { env::set_var("ASCSVCRT_SPRING__APPLICATION__NAME", "my-spring-app") };
+        let result = read_cloud_env();
+        assert_eq!(
+            result,
+            Some((
+                "my-spring-app".to_string(),
+                trace_utils::EnvironmentType::AzureSpringApp
+            ))
+        );
+    }
+
+    #[test]
+    fn test_no_environment_detected() {
+        let _lock = ENV_TEST_LOCK.lock().unwrap_or_else(|e| e.into_inner());
+        clear_all_env_vars();
+        let result = read_cloud_env();
+        assert_eq!(result, None);
+    }
+
+    #[test]
+    fn test_multiple_environments_returns_none() {
+        let _lock = ENV_TEST_LOCK.lock().unwrap_or_else(|e| e.into_inner());
+        clear_all_env_vars();
+        unsafe {
+            env::set_var("AWS_LAMBDA_INITIALIZATION_TYPE", "on-demand");
+            env::set_var("AWS_LAMBDA_FUNCTION_NAME", "my-lambda");
+            env::set_var("FUNCTIONS_EXTENSION_VERSION", "~4");
+            env::set_var("FUNCTIONS_WORKER_RUNTIME", "java");
+            env::set_var("WEBSITE_SITE_NAME", "my-azure-app");
+        }
+        let result = read_cloud_env();
+        assert_eq!(result, None);
+    }
+}