Make log messages clearer

Author: CVanF5

src/grpc.rs

@@ -586,7 +586,11 @@ pub fn epp_headers_blocking(
                     ngx_log_debug_http!(request, "ngx-inference: EPP returned no upstream");
                 }
                 Err(e) => {
-                    ngx_log_error_http!(request, "ngx-inference: EPP failed: {}", e);
+                    ngx_log_error_http!(
+                        request,
+                        "ngx-inference: EPP external service communication failed: {}",
+                        e
+                    );
                 }
             }
             grpc_result

src/lib.rs

@@ -708,7 +708,7 @@ http_request_handler!(inference_access_handler, |request: &mut http::Request| {
             }
             core::Status::NGX_ERROR => {
                 unsafe {
-                    let msg = b"ngx-inference: EPP processing failed\0";
+                    let msg = b"ngx-inference: EPP module processing failed internally\0";
                     ngx::ffi::ngx_log_error_core(
                         ngx::ffi::NGX_LOG_INFO as ngx::ffi::ngx_uint_t,
                         request.as_mut().connection.as_ref().unwrap().log,
@@ -725,7 +725,7 @@ http_request_handler!(inference_access_handler, |request: &mut http::Request| {
                             (*(*r_ptr).connection).log,
                             0,
                             #[allow(clippy::manual_c_str_literals)] // FFI code
-                            cstr_ptr(b"ngx-inference: EPP rejected request with HTTP 502 - external processor error\0".as_ptr()),
+                            cstr_ptr(b"ngx-inference: Module returning HTTP 502 due to EPP processing failure (fail-closed mode)\0".as_ptr()),
                         );
                     }
                     return http::HTTPStatus::BAD_GATEWAY.into();

src/modules/bbr.rs

@@ -309,7 +309,7 @@ unsafe fn read_request_body(
                         (*(*r).connection).log,
                         0,
                         #[allow(clippy::manual_c_str_literals)] // FFI code
-                        cstr_ptr(b"ngx-inference: BBR rejected request with HTTP 413 - actual payload size %uz exceeds limit %uz bytes\0".as_ptr()),
+                        cstr_ptr(b"ngx-inference: Module returning HTTP 413 - payload size %uz bytes exceeds BBR limit %uz bytes\0".as_ptr()),
                         total_read + len,
                         conf.bbr_max_body_size
                     );
@@ -347,7 +347,7 @@ unsafe fn read_request_body(
                         (*(*r).connection).log,
                         0,
                         #[allow(clippy::manual_c_str_literals)] // FFI code
-                        cstr_ptr(b"ngx-inference: BBR rejected request with HTTP 413 - actual payload size %uz exceeds limit %uz bytes\0".as_ptr()),
+                        cstr_ptr(b"ngx-inference: Module returning HTTP 413 - payload size %uz bytes exceeds BBR limit %uz bytes\0".as_ptr()),
                         total_read + file_size,
                         conf.bbr_max_body_size
                     );

src/modules/epp.rs

@@ -64,7 +64,7 @@ impl EppProcessor {
             Err(err) => {
                 ngx_log_info_http!(
                     request,
-                    "ngx-inference: EPP upstream selection failed: {}",
+                    "ngx-inference: EPP module failed to select upstream: {}",
                     err
                 );
                 if conf.epp_failure_mode_allow {
@@ -214,7 +214,11 @@ impl EppProcessor {
                 }
             }
             Err(err) => {
-                ngx_log_info_http!(request, "ngx-inference: EPP gRPC error: {}", err);
+                ngx_log_info_http!(
+                    request,
+                    "ngx-inference: EPP external service error: {}",
+                    err
+                );
                 return Err("epp grpc error");
             }
         }