Merge pull request 'Removed logs in sftp service' (#1187) from ranjith-dev into master
All checks were successful
Scan for leaked secrets using Kingfisher / kingfisher-secrets-scan (push) Successful in 14s
All checks were successful
Scan for leaked secrets using Kingfisher / kingfisher-secrets-scan (push) Successful in 14s
Reviewed-on: #1187
This commit was merged in pull request #1187.
This commit is contained in:
@@ -30,70 +30,24 @@ class ProcessSftpFiles extends Command
|
|||||||
public function handle()
|
public function handle()
|
||||||
{
|
{
|
||||||
|
|
||||||
Log::info('FTP disk config check', [
|
$files = Storage::disk('ftp')->allFiles();
|
||||||
'root' => config('filesystems.disks.ftp.root'),
|
|
||||||
'host' => config('filesystems.disks.ftp.host'),
|
|
||||||
'username' => config('filesystems.disks.ftp.username'),
|
|
||||||
'port' => config('filesystems.disks.ftp.port'),
|
|
||||||
]);
|
|
||||||
try{
|
|
||||||
|
|
||||||
$files = Storage::disk('ftp')->allFiles();
|
$folders = Storage::disk('ftp')->directories();
|
||||||
|
|
||||||
$folders = Storage::disk('ftp')->directories();
|
$allFiles = [];
|
||||||
|
|
||||||
$allFiles = [];
|
foreach ($folders as $folder) {
|
||||||
|
|
||||||
Log::info('FTP listing result', [
|
Log::info("Checking folder: $folder");
|
||||||
'files' => $files,
|
|
||||||
'file_count' => count($files),
|
|
||||||
'folders' => $folders,
|
|
||||||
'folder_count' => count($folders),
|
|
||||||
]);
|
|
||||||
|
|
||||||
foreach ($folders as $folder) {
|
// Get files inside the current folder
|
||||||
|
$files = Storage::disk('ftp')->files($folder);
|
||||||
|
|
||||||
Log::info("Checking folder: $folder");
|
foreach ($files as $file) {
|
||||||
|
|
||||||
try {
|
$allFiles[] = $file;
|
||||||
|
|
||||||
// Get files inside the current folder
|
|
||||||
$files = Storage::disk('ftp')->files($folder);
|
|
||||||
|
|
||||||
Log::info('FTP files response', [
|
|
||||||
'folder' => $folder,
|
|
||||||
'count' => count($files),
|
|
||||||
'files' => $files,
|
|
||||||
]);
|
|
||||||
|
|
||||||
foreach ($files as $file) {
|
|
||||||
Log::info("File: " . $file);
|
|
||||||
$allFiles[] = $file;
|
|
||||||
}
|
|
||||||
|
|
||||||
} catch (\Exception $e) {
|
|
||||||
|
|
||||||
// $this->error("Cannot access folder: " . $folder);
|
|
||||||
Log::info('FTP folder error', [
|
|
||||||
'folder' => $folder,
|
|
||||||
'error' => $e->getMessage(),
|
|
||||||
]);
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
catch (\Throwable $e) {
|
|
||||||
Log::info('FTP top-level error', [
|
|
||||||
'message' => $e->getMessage(),
|
|
||||||
'file' => $e->getFile(),
|
|
||||||
'line' => $e->getLine(),
|
|
||||||
'trace' => $e->getTraceAsString(),
|
|
||||||
]);
|
|
||||||
}
|
|
||||||
|
|
||||||
Log::info('Final allFiles', [
|
|
||||||
'count' => count($allFiles),
|
|
||||||
'files' => $allFiles,
|
|
||||||
]);
|
|
||||||
|
|
||||||
if (empty($allFiles)) {
|
if (empty($allFiles)) {
|
||||||
$this->info('No files to process.');
|
$this->info('No files to process.');
|
||||||
|
|||||||
Reference in New Issue
Block a user