[trace] Remove trace event from internal_dispatch_function

- The reason behind is because it might contain sensitive data
- Modify test that checks the number of expected logs from guest

Signed-off-by: Doru Blânzeanu <dblnz@pm.me>

Author: dblnz

src/hyperlight_common/src/flatbuffer_wrappers/function_call.rs

@@ -41,7 +41,7 @@ pub enum FunctionCallType {
 }
 
 /// `Functioncall` represents a call to a function in the guest or host.
-#[derive(Clone, Debug)]
+#[derive(Clone)]
 pub struct FunctionCall {
     /// The function name
     pub function_name: String,

src/hyperlight_guest_bin/src/guest_function/call.rs

@@ -91,9 +91,6 @@ pub(crate) fn internal_dispatch_function() {
         .try_pop_shared_input_data_into::<FunctionCall>()
         .expect("Function call deserialization failed");
 
-    #[cfg(debug_assertions)]
-    tracing::trace!("{:?}", function_call);
-
     let res = call_guest_function(function_call);
 
     match res {

src/hyperlight_host/tests/integration_test.rs

@@ -744,14 +744,10 @@ fn log_message() {
     //  - internal_dispatch_function does a log::trace! in debug mode
     //  - logs from trace level tracing spans created as logs because of the tracing `log` feature
     //    - 4 from evolve call (hyperlight_main + halt)
-    //    - 8 from guest call (internal_dispatch_function + others)
+    //    - 7 from guest call
     // and are multiplied because we make 6 calls to `log_test_messages`
     // NOTE: These numbers need to be updated if log messages or spans are added/removed
-    let num_fixed_trace_log = if cfg!(debug_assertions) {
-        (1 + 12) * 6
-    } else {
-        12 * 6
-    };
+    let num_fixed_trace_log = 12 * 6;
 
     let tests = vec![
         (LevelFilter::Trace, 5 + num_fixed_trace_log),