Fix: Changing the default event_logging value to 'false' and ensuring that, if true, it logs at debug level instead of info.

Author: JeremieRodon

lambda-appsync-proc/src/appsync_lambda_main/mod.rs

@@ -170,7 +170,7 @@ impl Default for OptionalParameters {
             hook: None,
             log_init: None,
             #[cfg(feature = "log")]
-            event_logging: true,
+            event_logging: false,
             tos: TypeOverrides::new(),
             nos: NameOverrides::new(),
         }
@@ -329,7 +329,7 @@ impl AppsyncLambdaMain {
         #[cfg(feature = "log")]
         if self.options.event_logging {
             log_lines.extend(quote! {
-                ::lambda_appsync::log::info!("event={event:?}");
+                ::lambda_appsync::log::debug!("event={event:?}");
             });
         }
         #[cfg(feature = "log")]
@@ -450,8 +450,7 @@ impl AppsyncLambdaMain {
         #[cfg(feature = "log")]
         if self.options.event_logging {
             log_lines.extend(quote! {
-                ::lambda_appsync::log::debug!("{event:?}");
-                ::lambda_appsync::log::info!("{}", ::lambda_appsync::serde_json::json!(event.payload));
+                ::lambda_appsync::log::debug!("{}", ::lambda_appsync::serde_json::json!(event.payload));
             });
         }
 

lambda-appsync-proc/src/lib.rs

@@ -29,8 +29,9 @@ use proc_macro::TokenStream;
 ///
 /// - `batch = bool`: Enable/disable batch request handling (default: true)
 /// - `hook = fn_name`: Add a custom hook function for request validation/auth
-/// - (feature: `log`) `log_init = fn_name`: Use a custom log initialization function instead of the default one
-/// - (feature: `log`) `event_logging = bool`: If true, the macro will generate code to dump the entire event in the logs (default: `true`)
+/// - `log_init = fn_name`: Use a custom log initialization function instead of the default one
+/// - (feature: `log`) `event_logging = bool`: If true, the macro will generate code to dump the
+///   lambda payload JSON as well as parsed `AppsyncEvent<Operation>`s in the logs at debug level (default: `false`)
 /// - `exclude_lambda_handler = bool`: Skip generation of Lambda handler code
 /// - `only_lambda_handler = bool`: Only generate Lambda handler code
 /// - `exclude_appsync_types = bool`: Skip generation of GraphQL type definitions

lambda-appsync-proc/tests/pass/event_logging_option.rs

@@ -1,6 +1,6 @@
 mod no_run {
-    // Test with event_logging disabled
-    lambda_appsync::appsync_lambda_main!("../../../../schema.graphql", event_logging = false);
+    // Test with event_logging enabled
+    lambda_appsync::appsync_lambda_main!("../../../../schema.graphql", event_logging = true);
 }
 
 fn main() {}