Skip to content

Fix #5104 : Ne pas lancer un Job si le décodage des arguments n’est pas correct

cpol requested to merge gh-3abd0771/5625/unknown/refs/pull/5625/head into master

Hello, sans réponse dans le ticket je me permet de pousser ma PR. J'ai hésité à garder le ticket d'origine, car la stratégie ici est légèrement différente: je ne cherche pas à essayer de deviner le nombre d'arguments à obtenir, je pars du principe que si ça foire il faut arrêter le traitement et tuer le job proprement. On peut faire un nouveau ticket si besoin.

Testé sur une 4.1.9 et 4.1.10

Reproduction du bug et validation: cf ticket:

  • se débrouiller pour obtenir la génération d'un cache 404 avec un emoji dans l'url.
  • trace Pub:info: arguments job errones visible dans queue.log
  • ️ pas de call stack
  • ️ job suprimé dans la liste des tâches
  • ️ plus de trace Pub:ERREUR: queue_close_job car _JQ_PENDING depuis +180s

Merge request reports