data/bericht
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix: Cronjob entfernt, Cleanup opportunistisch beim Token-Create
All checks were successful
Deploy bericht / deploy (push) Successful in 1s
Details

Browse source 
Der Dolibarr-Cron-Scheduler ist auf Prod nicht aktiv, deshalb hing
der tägliche Cleanup-Job 'Expired Upload-Tokens bereinigen' auf
'Geplant' und GlobalNotify meldete ihn als hängenden Job.

Lösung: Cronjob komplett aus dem Modul entfernt. Das Cleanup läuft
jetzt opportunistisch bei jedem neuen Token-Insert:

BerichtUploadToken::create() führt vor dem INSERT ein
DELETE FROM llx_bericht_upload_token WHERE expires_at < NOW()
aus. Das ist minimal teurer (1 Query extra pro Token-Create,
~0ms bei leerer Tabelle), aber vollständig cron-unabhängig.

WICHTIG für Prod: Nach Deploy muss der bestehende Cron-Eintrag
manuell aus llx_cronjob gelöscht werden (oder das Modul einmal
deaktiviert + reaktiviert werden, dann wird er mit remove() bzw.
init() neu gesetzt — ohne Eintrag). GlobalNotify ist danach ruhig.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
[deploy]
This commit is contained in:
Eduard Wisch 2026-04-09 08:58:39 +02:00
parent 344f884a0f
commit 1705744809
2 changed files with 8 additions and 17 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
class/upload_token.class.php
View file

@ -25,10 +25,15 @@ class BerichtUploadToken
/** /**
* Erstellt einen neuen Token für einen Bericht. * Erstellt einen neuen Token für einen Bericht.
* Räumt dabei abgelaufene Tokens gleich mit auf (kein Cronjob nötig).
* @return string|false Hex-Token bei Erfolg * @return string|false Hex-Token bei Erfolg
*/ */
public function create($fk_bericht, $fk_user, $lifetime = null, $max_uploads = null) public function create($fk_bericht, $fk_user, $lifetime = null, $max_uploads = null)
{ {
// Opportunistisches Cleanup: entferne abgelaufene Tokens bei jedem Insert
$this->db->query("DELETE FROM ".$this->db->prefix()."bericht_upload_token"
." WHERE expires_at < '".$this->db->idate(dol_now())."'", 1);
$this->token = bin2hex(random_bytes(32)); $this->token = bin2hex(random_bytes(32));
$this->fk_bericht = (int) $fk_bericht; $this->fk_bericht = (int) $fk_bericht;
$this->fk_user_creat = (int) $fk_user; $this->fk_user_creat = (int) $fk_user;

20
core/modules/modBericht.class.php
View file

@ -89,23 +89,9 @@ class modBericht extends DolibarrModules
$this->dictionaries = array(); $this->dictionaries = array();
$this->boxes = array(); $this->boxes = array();
// Cleanup expired Mobile-Upload-Tokens (täglich um 03:30) // Kein Cronjob — Cleanup expired Upload-Tokens passiert on-demand
$this->cronjobs = array( // beim Anlegen eines neuen Tokens (siehe BerichtUploadToken::create()).
0 => array( $this->cronjobs = array();
'label' => 'Bericht: Expired Upload-Tokens bereinigen',
'jobtype' => 'method',
'class' => '/bericht/class/upload_token.class.php',
'objectname' => 'BerichtUploadToken',
'method' => 'cleanupExpired',
'parameters' => '',
'comment' => 'Löscht abgelaufene Mobile-Upload-Tokens aus llx_bericht_upload_token',
'frequency' => 1,
'unitfrequency' => 86400,
'status' => 1,
'test' => 'isModEnabled("bericht")',
'priority' => 50,
),
);
// Rechte — wie Stundenzettel: [4]=perms, [5]=subperms (leer) // Rechte — wie Stundenzettel: [4]=perms, [5]=subperms (leer)
$this->rights = array(); $this->rights = array();