[SPIP - Anomalie #4345] super_cron HS en https

[email protected]
Newsgroups gmane.comp.web.spip.devel
Message-ID <[email protected]>
La demande #4345 a été mise à jour par jluc -.


Pour produire un diff à intégrer directement il me faudrait savoir ce que je met dedans : tout ? Sinon c'est en kit :
- le fix pour 80 / 443 via les constantes et en testant le scheme comme queue_affichage_cron ? (a priori oui !)
- l'emploi d'un timestamp ?
- le timestamp divisé qui introduit un délai ?
- le remplacement des commentaires phpdoc comme indiqué plus haut ? https://core.spip.net/issues/4345#note-8
- les logs dans les divers cas d'erreur déjà testé ?
- le test du statut de retour du fsockopen en plus, avec log si c'est pas 2XX ?
- la correction de queue_affichage_cron pour employer les constantes ?

----------------------------------------
Anomalie #4345: super_cron HS en https
https://core.spip.net/issues/4345#change-15027

* 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
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.