I due piu' gravi potevano causare perdita di dati silenziosa. AppServiceProvider chiamava key:generate --force a ogni richiesta quando APP_KEY risultava vuota. Poiche' nome e cognome dei proclamatori sono cifrati con quella chiave, ogni riga scritta sotto una chiave poi sostituita diventava irrecuperabile. La chiave e' ora risolta una sola volta all'avvio del container e il provider solleva un'eccezione se manca, invece di rigenerarla. importTerritoryPdfFolder() caricava i PDF, costruiva l'elenco dei file e poi lo scartava senza mai accodare il job: nessun errore, nessun territorio aggiornato, e i file restavano su disco a crescita illimitata perche' la pulizia viveva solo nel finally del job mai eseguito. Sicurezza: - i permessi non venivano riapplicati alle richieste /livewire/update, per cui un permesso revocato restava attivo fino al reload della pagina - host header injection: nginx passava l'Host del client come X-Forwarded-Host e Laravel si fidava di tutti i proxy, abbastanza per far generare a route() link verso domini arbitrari, inclusi i PDF pubblici che contengono un codice di accesso valido - lo schema forwarded e' ora deciso dal server e non dall'header del client - campagna_id in Rientra e' #[Locked]: era derivato lato server ma manipolabile dal client, permettendo di conteggiare un rientro su una campagna arbitraria - sortField del registro passava non validato a orderBy(), con 500 su colonna inesistente - SESSION_SECURE_COOKIE ora vale true fuori da local - limiti su dimensione e numero di voci negli import ZIP Integrita' e prestazioni: - il default di assignment_link_ttl_hours era rimasto 24 dopo la conversione da ore a mesi: i link PDF pubblici duravano due anni invece di uno mese - il seeder usava syncPermissions(), che a un secondo giro azzerava i permessi personalizzati dei ruoli; ora e' additivo - pdftoppm senza timeout poteva bloccare il worker indefinitamente - nomi file deterministici nell'import: "1 A" e "1-A" scrivevano sullo stesso PDF, sovrascrivendosi - Setting::flushCache() dopo le cancellazioni massive, che non emettono eventi e lasciavano la cache puntata a una riga inesistente - riga settings creata dal seeder, per evitare due singleton concorrenti - rimosse due migrazioni ridondanti che rompevano migrate:rollback - indici su assigned_at, (attivo, deleted_at) e deleted_at - risolti gli N+1 negli elenchi territori e proclamatori Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
126 lines
4.4 KiB
PHP
126 lines
4.4 KiB
PHP
<?php
|
|
|
|
namespace App\Services;
|
|
|
|
use App\Jobs\ImportTerritoryPdfFolder;
|
|
use Illuminate\Http\UploadedFile;
|
|
use Illuminate\Support\Facades\Storage;
|
|
use Illuminate\Support\Str;
|
|
use RuntimeException;
|
|
use ZipArchive;
|
|
|
|
class TerritorioPdfImportDispatcher
|
|
{
|
|
/** Per-entry uncompressed ceiling, matching the 10 MB Livewire upload rule. */
|
|
private const MAX_ENTRY_BYTES = 10 * 1024 * 1024;
|
|
|
|
/** Caps the job payload — every entry is serialized into one queue message. */
|
|
private const MAX_ENTRIES = 2000;
|
|
|
|
public function __construct(
|
|
protected TerritorioPdfImportState $stateService,
|
|
) {
|
|
}
|
|
|
|
public function dispatchStoredFiles(string $importId, array $storedFiles, ?int $actorId, string $initialLog): array
|
|
{
|
|
$state = $this->stateService->initialize($importId, count($storedFiles));
|
|
$this->stateService->appendLog($importId, $initialLog);
|
|
|
|
ImportTerritoryPdfFolder::dispatch($importId, $storedFiles, $actorId);
|
|
|
|
return $this->stateService->get($importId) ?? $state;
|
|
}
|
|
|
|
public function dispatchUploadedZip(UploadedFile $zipFile, ?int $actorId): string
|
|
{
|
|
$importId = (string) Str::uuid();
|
|
$zipStoredPath = $zipFile->storeAs(
|
|
'bulk-territori-imports/' . $importId,
|
|
'archivio-' . $importId . '.zip',
|
|
'local'
|
|
);
|
|
|
|
$zipAbsolutePath = storage_path('app/' . $zipStoredPath);
|
|
$zip = new ZipArchive();
|
|
|
|
if ($zip->open($zipAbsolutePath) !== true) {
|
|
throw new RuntimeException('Impossibile aprire il file ZIP.');
|
|
}
|
|
|
|
$storedFiles = [];
|
|
$entryIndex = 0;
|
|
|
|
try {
|
|
for ($index = 0; $index < $zip->numFiles; $index++) {
|
|
$entryName = $zip->getNameIndex($index);
|
|
|
|
if (! $entryName || str_ends_with($entryName, '/')) {
|
|
continue;
|
|
}
|
|
|
|
if (strtolower(pathinfo($entryName, PATHINFO_EXTENSION)) !== 'pdf') {
|
|
continue;
|
|
}
|
|
|
|
if ($entryIndex >= self::MAX_ENTRIES) {
|
|
throw new RuntimeException(
|
|
'Lo ZIP contiene più di ' . self::MAX_ENTRIES . ' PDF. Suddividilo in archivi più piccoli.'
|
|
);
|
|
}
|
|
|
|
// Check the *uncompressed* size before reading. getFromIndex()
|
|
// inflates the entry entirely into memory, so a small zip with
|
|
// one hugely compressible entry would otherwise exhaust PHP's
|
|
// memory limit and abort the request mid-extraction.
|
|
$uncompressedSize = $zip->statIndex($index)['size'] ?? 0;
|
|
if ($uncompressedSize > self::MAX_ENTRY_BYTES) {
|
|
throw new RuntimeException(sprintf(
|
|
'Il file "%s" nello ZIP supera %d MB.',
|
|
basename($entryName),
|
|
self::MAX_ENTRY_BYTES / 1048576
|
|
));
|
|
}
|
|
|
|
$content = $zip->getFromIndex($index);
|
|
if ($content === false) {
|
|
continue;
|
|
}
|
|
|
|
$originalName = basename($entryName);
|
|
$safeName = Str::slug(pathinfo($originalName, PATHINFO_FILENAME));
|
|
$storedPath = 'bulk-territori-imports/' . $importId . '/zip-' . str_pad((string) $entryIndex, 4, '0', STR_PAD_LEFT) . '-' . $safeName . '.pdf';
|
|
|
|
file_put_contents(storage_path('app/' . $storedPath), $content);
|
|
|
|
$storedFiles[] = [
|
|
'original_name' => $originalName,
|
|
'stored_path' => $storedPath,
|
|
];
|
|
|
|
$entryIndex++;
|
|
}
|
|
} catch (\Throwable $exception) {
|
|
// Nothing was queued, so the job's own cleanup will never run —
|
|
// drop the partially extracted files rather than orphaning them.
|
|
Storage::disk('local')->deleteDirectory('bulk-territori-imports/' . $importId);
|
|
|
|
throw $exception;
|
|
} finally {
|
|
$zip->close();
|
|
}
|
|
|
|
if ($storedFiles === []) {
|
|
throw new RuntimeException('Lo ZIP non contiene file PDF validi.');
|
|
}
|
|
|
|
$this->dispatchStoredFiles(
|
|
$importId,
|
|
$storedFiles,
|
|
$actorId,
|
|
'Archivio ZIP ricevuto: ' . count($storedFiles) . ' PDF estratti e messi in coda per l\'elaborazione.'
|
|
);
|
|
|
|
return $importId;
|
|
}
|
|
} |