[SPIP - Anomalie #4345] (En cours) super_cron HS en https
| Newsgroups | gmane.comp.web.spip.devel |
|---|---|
| Message-ID | <[email protected]> |
La demande #4345 a été mise à jour par b b. Statut changé de Nouveau à En cours Le patch grossit à vue d'oeil, pourquoi ajouter une nouvelle constante @_SUPER_CRON_DELAIS@ ? De plus, il serait intéressant de s'inspirer de ce que fait https://core.spip.net/projects/spip/repository/entry/spip/ecrire/inc/queue.php#L630 pour la détection du port. Et pour finir, ça serait pas mal de respecter les PSRs dans tes structures de test :) ---------------------------------------- Anomalie #4345: super_cron HS en https https://core.spip.net/issues/4345#change-15025 * Auteur: jluc - * Statut: En cours * Priorité: Normal * Assigné à: * Catégorie: * Version cible: * Resolution: * Navigateur: ---------------------------------------- action_super_cron_dist appelle fsockopen avec le port 80 par défaut, si l'url issue de genererl_url_action ne spécifie pas le port (c'est à dire la plupart du temps). sur un site en https ça échoue (éventuellement ça renvoie une 301 mais fsockopen ne les suit pas) Faute de parvenir à proposer une PR git, voici un patch à l'ancienne qui corrige ça en testant si http ou https. En plus, il loge les échecs de fsockopen afin de pas avoir une partie silencieuse de SPIP qui ne marche pas. Et il teste aussi le statut renvoyé. Cette partie peut être supprimée si nécessaire. ---Fichiers----------------------------- supercron.diff (1,18 ko) supercron_simple.diff (1002 octets) supercron v2.diff (1,41 ko) -- Vous recevez ce mail car vous êtes impliqués sur ce projet. Pour changer les préférences d'envoi de mail, allez sur http://core.spip.net/my/account