Changeset 796 for trunk

Timestamp:

Jul 13, 2023 3:13:57 AM (10 months ago)

Author:

anonymous

Message:

minor changes

Location:

trunk/lib

Files:

• App.inc.php (modified) (1 diff)
• Utilities.inc.php (modified) (2 diffs)

trunk/lib/App.inc.php

@@ -446 +446 @@
                     }
                 } else {
-                    $this->logMsg(sprintf('DB auth file not found: %s', $this->getParam('db_auth_file')), LOG_ERR, __FILE__, __LINE__);
+                    $missing_db_params = [];
+                    foreach (['db_auth_file', 'db_server', 'db_name', 'db_user', 'db_pass'] as $required_db_param) {
+                        if (!$this->getParam($required_db_param)) {
+                            $missing_db_params[] = $required_db_param;
+                        }
+                    }
+                    $this->logMsg(sprintf('DB credentials or %s not found. Missing parameters: %s', $this->getParam('db_auth_file'), join(', ', $missing_db_params)), LOG_ERR, __FILE__, __LINE__);
                 }
             }

trunk/lib/Utilities.inc.php

@@ -1736 +1736 @@
     $headers = get_headers($url, 1, $context);
     $app =& \App::getInstance();
-    $app->logMsg(getDump($headers, true, SC_DUMP_JSON), LOG_DEBUG, __FILE__, __LINE__);
+    $app->logMsg(sprintf('HTTP response headers for %s %s: %s', strtoupper($method), $url, getDump($headers, true, SC_DUMP_JSON)), LOG_DEBUG, __FILE__, __LINE__);
     if (empty($headers)) {
         return false;
@@ -1745 +1745 @@
     $final_http_status_key = end($http_status_keys); // E.g., `1`
     $final_http_status = $headers[$final_http_status_key]; // E.g., `HTTP/1.1 200 OK`
-    $app->logMsg(sprintf('$final_http_status: %s', $final_http_status), LOG_DEBUG, __FILE__, __LINE__);
+    $app->logMsg(sprintf('Last HTTP status code: %s', $final_http_status), LOG_DEBUG, __FILE__, __LINE__);
     if ($headers && preg_match(sprintf('/\b(%s)\b/', join('|', $valid_response_codes)), $final_http_status)) {
         $headers = array_change_key_case($headers, CASE_LOWER);