[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 -.


Il y a en effet plusieurs trucs : une correction de code, une amélioration potentielle et des modifs des commentaires.

La constante _SUPER_CRON_DELAIS permet d'ajouter un timestamp sur l'appel de cron. Cela vise à ce que le php reçoive vraiment la requête au lieu qu'elle soit interceptée par un varnish "mal configuré" (comme les timestamp qu'on ajoute aux fichiers images)
Dans le cas où il y a un cache de ce type et si la valeur de _SUPER_CRON_DELAIS vaut plus que 1, cela divise d'autant la fréquence des appels au cron en cas de grosse fréquentation. 
Si ça convient pas comme ça, on pourrait la garder mais ne pas diviser le timestamp avec (et alors,  la renommer _SUPER_CRON_TIMESTAMP )

Autre point :
il faudrait utiliser les constantes _PORT_HTTP_STANDARD et _PORT_HTTPS_STANDARD si définies plutôt que 80 et 443, 
ici et aussi ailleurs : dans la fonction queue_affichage_cron 

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

* 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.