[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,11 +744,11 @@ 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
+        (1 + 11) * 6
     } else {
         12 * 6
     };