Fixed Data Sanisation issues.

Author: colinmurphy

plugins/wpgraphql-logging/src/Admin/View/List/ListTable.php

@@ -392,7 +392,7 @@ public function get_memory_usage(WordPressDatabaseEntity $item): string {
 	public function get_request_headers(WordPressDatabaseEntity $item): string {
 		$extra           = $item->get_extra();
 		$request_headers = $extra['request_headers'] ?? [];
-		if ( ! is_array( $request_headers ) ) {
+		if ( ! is_array( $request_headers ) || empty( $request_headers ) ) {
 			return '';
 		}
 

plugins/wpgraphql-logging/src/Logger/Processors/DataSanitizationProcessor.php

@@ -124,33 +124,20 @@ protected function apply_rule(array &$data, string $key, string $rule): void {
 
 		$keys     = explode( '.', $key );
 		$last_key = array_pop( $keys );
-		$current  = $this->navigate_to_parent( $data, $keys );
 
-		if ( null === $current || ! array_key_exists( $last_key, $current ) ) {
-			return;
-		}
-
-		$this->apply_sanitization_rule( $current, $last_key, $rule );
-	}
-
-	/**
-	 * Navigate to the parent array of the target key.
-	 *
-	 * @param array<string, mixed> $data The data array to navigate.
-	 * @param array<string>        $keys The keys to navigate through.
-	 *
-	 * @return array<string, mixed>|null The parent array or null if not found.
-	 */
-	protected function &navigate_to_parent(array &$data, array $keys): ?array {
 		$current = &$data;
 		foreach ( $keys as $segment ) {
 			if ( ! is_array( $current ) || ! isset( $current[ $segment ] ) ) {
-				$null = null;
-				return $null;
+				return;
 			}
 			$current = &$current[ $segment ];
 		}
-		return $current;
+
+		if ( ! is_array( $current ) || ! array_key_exists( $last_key, $current ) ) {
+			return;
+		}
+
+		$this->apply_sanitization_rule( $current, $last_key, $rule );
 	}
 
 	/**
@@ -199,8 +186,8 @@ public function __invoke( LogRecord $record ): LogRecord {
 			return $record;
 		}
 
-		$context = $record['context'] ?? [];
-		$extra   = $record['extra'] ?? [];
+		$context = $record->context;
+		$extra   = $record->extra;
 		foreach ( $rules as $key => $rule ) {
 			$this->apply_rule( $context, $key, $rule );
 			$this->apply_rule( $extra, $key, $rule );