diff --git a/components/udf/hooks/logging_hook.h b/components/udf/hooks/logging_hook.h index e681c46d..f549d1dd 100644 --- a/components/udf/hooks/logging_hook.h +++ b/components/udf/hooks/logging_hook.h @@ -38,7 +38,7 @@ inline void LogMessage( return; } PS_VLOG(10, request_context->GetPSLogContext()) << "Called logging hook"; - PS_LOG(INFO, request_context->GetPSLogContext()) + PS_VLOG(1, request_context->GetPSLogContext()) << payload.io_proto.input_string(); } diff --git a/components/udf/udf_client_test.cc b/components/udf/udf_client_test.cc index 9f447ae6..d3c240a5 100644 --- a/components/udf/udf_client_test.cc +++ b/components/udf/udf_client_test.cc @@ -72,6 +72,7 @@ absl::StatusOr> CreateUdfClient() { class UdfClientTest : public ::testing::Test { protected: UdfClientTest() { + privacy_sandbox::server_common::log::SetGlobalPSVLogLevel(1); privacy_sandbox::server_common::log::ServerToken( kExampleConsentedDebugToken); const std::string telemetry_config_str = R"pb( diff --git a/public/udf/constants.h b/public/udf/constants.h index cd29376f..e33e48db 100644 --- a/public/udf/constants.h +++ b/public/udf/constants.h @@ -54,7 +54,6 @@ function handlePas(udf_arguments) { if (udf_arguments.length != 1) { const error_message = 'For PAS default UDF exactly one argument should be provided, but was provided ' + udf_arguments.length; - logMessage(error_message); throw new Error(error_message); } const kv_result = JSON.parse(getValues(udf_arguments[0])); @@ -63,7 +62,6 @@ function handlePas(udf_arguments) { } const error_message = "Error executing handle PAS:" + JSON.stringify(kv_result); - logMessage(error_message); throw new Error(error_message); } @@ -73,10 +71,8 @@ function handlePA(udf_arguments) { } function HandleRequest(executionMetadata, ...udf_arguments) { - logMessage("Executing UDF"); if(executionMetadata.requestMetadata && executionMetadata.requestMetadata.is_pas) { - logMessage('Executing PAS branch'); return handlePas(udf_arguments); } return handlePA(udf_arguments);