DMeloni / shaarlo

Unified Shaarlis Rss
33 stars 9 forks source link

[Bug] "Proposition" : erreur avec les Shaarlis en SSL #19

Closed Draky50110 closed 10 years ago

Draky50110 commented 10 years ago

Salut

J'ai testé la "Proposition de flux".

Il m'en trouve (il utilise quoi comme méthode ? Google ?) mais dès qu'un flux est en SSL, il merdoit :)

Le résultat avec les messages d'erreur : http://gilles.wittezaele.fr/paste/?f8579b57286cb863#lmPy89XUAkh3MJpcy2kIp/PdwBTDaS0ALeKVbfL8NcY

Voilà :)

DMeloni commented 10 years ago

Hello,

a priori, ce ne sont que des warnings tous justifiés (quand on interroge l'url directement via un navigateur, on se tape également une erreur).

Le seul hic que je vois c'est que les warnings sont affichés. Trois solutions dans ce cas : -modifier la config php.ini en passant : //Config en Production display_errors = Off OU error_reporting = E_ERROR & ~E_DEPRECATED & ~E_STRICT

-modifier la fonction urlExists du fct_rss.php en mettant un @ devant get_headers Le truc c'est que cette solution évite l'erreur mais la cache encore plus donc bonjour la phase de débug

-modifier la page admin.php en indiquant un error_reporting à 0, ça te permet de garder ta conf php et ça protège le code. C'est un peu crade mais c'est une solution un peu propre que l'utilisation du @ et moins ennuyante pour l'utilisateur (toi).

-> je commit ça

...

EDIT : admin.php a déjà un error_reporting à 0, donc ton erreur ne devrait en théorie pas s'afficher, tu peux afficher l'entete (les 10 premieres lignes) du fichier admin.php que tu as ?

Draky50110 commented 10 years ago

Voilà le début de "admin.php" : http://gilles.wittezaele.fr/paste/?42f310ec05998bb9#RTpLi2GsY4UBzItCPT95fic6FQtdS2KWTNl5GKh1EEc

DMeloni commented 10 years ago

Ok, ça vient donc du module de proposition. C'est corrigé :-)

Draky50110 commented 10 years ago

:+1: