From 869eb99e707d61ff0e74778b969ec3b2e961b133 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=E3=81=AE=E3=81=B6?= Date: Tue, 3 Mar 2026 10:34:54 +0900 Subject: [PATCH] =?UTF-8?q?fix:=20error=5Flog=E3=81=8B=E3=82=89CSV?= =?UTF-8?q?=E8=A1=8C=E3=83=87=E3=83=BC=E3=82=BF(PII=E5=90=AB=E3=82=80)?= =?UTF-8?q?=E3=81=AE=E5=87=BA=E5=8A=9B=E3=82=92=E9=99=A4=E5=8E=BB?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit saveToC()とconvertDataTypesForPostgreSQL()のcatchブロックで json_encode($data)/$json_encode($value)をerror_logに出力していた。 CSVには顧客パスワードハッシュ・メールアドレス等が含まれるため、 ログファイルにPIIが永続するリスクがあった。 テーブル名・行番号・例外メッセージのログは維持し、 デバッグに必要な情報は確保しつつPII漏洩を防止する。 Co-Authored-By: Claude Opus 4.6 --- Controller/Admin/ConfigController.php | 2 -- Service/DataMigrationService.php | 1 - 2 files changed, 3 deletions(-) diff --git a/Controller/Admin/ConfigController.php b/Controller/Admin/ConfigController.php index 33f2872..754c1ee 100644 --- a/Controller/Admin/ConfigController.php +++ b/Controller/Admin/ConfigController.php @@ -436,8 +436,6 @@ private function saveToC($em, $tmpDir, $csvName, $tableName = null, $allow_zero $builder->execute(); } catch (\Exception $e) { error_log("BulkInsertQuery execute error in saveToC table '$tableName' at row $i: " . $e->getMessage()); - error_log("Failed data for row $i: " . json_encode($value)); - error_log("Original CSV data: " . json_encode($data)); throw $e; } } diff --git a/Service/DataMigrationService.php b/Service/DataMigrationService.php index 8493e03..b3d0069 100644 --- a/Service/DataMigrationService.php +++ b/Service/DataMigrationService.php @@ -203,7 +203,6 @@ public function convertDataTypesForPostgreSQL($em, $tableName, $data) } } catch (\Exception $e) { error_log("Error in convertDataTypesForPostgreSQL for table '$tableName': " . $e->getMessage()); - error_log("Data being processed: " . json_encode($data)); // エラーが発生した場合は元のデータをそのまま返す }