tontof / kriss_feed

A simple and smart (or stupid) feed reader
280 stars 53 forks source link

Feed not working: http://www.cyrille-borne.com/index.php?feed/atom #49

Closed sebsauvage closed 11 years ago

sebsauvage commented 11 years ago

I cannot add this feed: http://www.cyrille-borne.com/index.php?feed/atom

Although it seem to be valid: http://validator.w3.org/feed/check.cgi?url=http%3A%2F%2Fwww.cyrille-borne.com%2Findex.php%3Ffeed%2Fatom

tontof commented 11 years ago

No problem for me : http://tontof.net/feed/?currentHash=OPxdgA Try again ?

ghost commented 11 years ago

Ca va pas arranger mes problèmes de mégalomanie, excusez moi j'ai un billet à faire d'urgence :D

tontof commented 11 years ago

@sebsauvage : tu as ressayé pour voir si ça marchait ?

sebsauvage commented 11 years ago

ça ne marche toujours pas, mais attends je vais ré-essayer avec le nouveau master.

sebsauvage commented 11 years ago

mmm... non ça ne marche toujours pas: "The feed you are trying to add already exists or is wrong. Check your feed or try again later."

(et je viens de vérifier, le flux est valide).

Pour info sur mon serveur: PHP Version 5.3.8 libxml Version 2.7.7 SimpleXML: Revision 314376

tontof commented 11 years ago

Tu as essayé avec https://github.com/tontof/kriss_feed/raw/master/src/index.php ? ou https://github.com/tontof/kriss_feed/raw/master/index.php

Arf j'ai peur que ce soit libxml, moi c'est 2.7.8...

sebsauvage commented 11 years ago

J'ai pris le fichier https://raw.github.com/tontof/kriss_feed/master/index.php il y a quelques minutes.

tontof commented 11 years ago

En fait la dernière c'est : https://github.com/tontof/kriss_feed/raw/master/src/index.php l'autre c'est toujours la version 5

sebsauvage commented 11 years ago

oups... j'ai plein de warnings avec cette version: "Warning: strftime() [function.strftime]: It is not safe to rely on the system's timezone settings. You are required to use the date.timezone setting or the date_default_timezone_set() function. In case you used any of those methods and you are still getting this warning, you most likely misspelled the timezone identifier. We selected 'Europe/Berlin' for 'CET/1.0/no DST' instead in /home/.../index.php on line 4329"

Il faudrait un petit date_default_timezone_set('Europe/Paris'); en début de script.

sebsauvage commented 11 years ago

Non ça ne marche pas plus avec cette version. Même message d'erreur.

dacrovinunghi commented 11 years ago

Excuser je fais que passer ca marche pas pour moi non plus dans l'autre issu. Big respect à toi seb

tontof commented 11 years ago

Arf c'est l'ajout de la date qui pose problème je vais encore devoir copier du code de shaarli :-) C'est similaire à https://github.com/tontof/kriss_feed/issues/47

Du coup si ça ne marche pas, je pense que c'est libxml, je vais essayer de downgrader pour voir si ça fait pareil

ghost commented 11 years ago

Je fais quand même honneur à ma réputation de type qui sort des bugs en pagaille, même sans le vouloir. Est ce qu'avec ça http://www.cyrille-borne.com/index.php?feed/rss2 ça passe ? Sur mon serveur Wheezy pas de souci.

sebsauvage commented 11 years ago

@cborne: pas mieux avec /rss2.

tontof commented 11 years ago

En même temps est-ce que ce flux est indispensable ? :-p

(J'ai peur de devoir utiliser un autre parser XML parce que libxml semble ne pas être bien à jour sur les serveurs)

sebsauvage commented 11 years ago

C'est pas bien grave :-)

(Purée GitHub est rudement bien: Les commentaires se mettent à jour en temps réel.)

tontof commented 11 years ago

Ouais mais c'est propriétaire, hein Cyrille ? ^^

ghost commented 11 years ago

Faudrait voir éventuellement si avec un autre flux dotclear ça pose problème, le site de tristan nitot c'est du dotclear. Et oui ce flux est indispensable, c'est une évidence :D

sebsauvage commented 11 years ago

Des fois c'est chiant les services trop bien faits: ça devient comme une drogue. Faut se méfier de ceux qui dealent de la trop bonne dope.

Et je préfère avoir la recette de la dope pour ne pas être en manque: Vive l'opensource. (Purée, je réalise que google l'a très bien joué: Ils ont habitué tout le monde avec l'un des meilleurs lecteurs RSS en ligne, gratuit, et pouf, il sèvrent tout le monde en espérant qu'ils choisiront la nouvelle drogue, Google+)

ghost commented 11 years ago

En fait tontof a collé un filtre sur mon blog parce que je parle plus de Leed que de Kriss Feed, c'est pas un bug :D

tontof commented 11 years ago

It's not a bug it's a feature :-)

JeromeJ commented 11 years ago

Je conseillerais à sebsauvage d'essayer de momentanément désactiver le passage sous silence des exceptions pouvant être levée par set_error_handler(array('MyTool', 'silence_errors')); qui se situe ligne 4476 chez moi dans la fonction Feed->loadXml.

En tout cas chez moi ça m'a permis de voir d'où venait mon prob (moi c'était openssl qui n'était pas activé de mon côté: https://github.com/tontof/kriss_feed/issues/59)

tontof commented 11 years ago

À confirmer, mais généralement l'erreur obtenue quand le flux est valide c'est un truc de ce style : Warning: DOMDocument::load(): Start tag expected, '<' not found Pas très explicite malheureusement.

J'ai essayé de compiler libxml2.so.2.7.7 et de remplacer le lien /usr/lib/i386-linux-gnu/libxml2.so.2 Et quand j'ai testé avec le flux de Cyrille ça a marché mais comme j'ai pas touché à SimpleXML c'est difficile encore de tirer des conclusions :-( Je pense que ça a effectivement utilisé la version 2.7.7 mais ça affiche toujours 2.7.8 dans phpinfo.

Du coup je suis en train d'essayer de compiler php pour confirmer.

jcb commented 11 years ago

Chez moi le flux passe nickel, et voici ce qui est installé:

tontof commented 11 years ago

@jcb : J'ai ajouté ton fix hier soir. Comme tu as une config de libxml récente, tu veux bien essayer http://www.altdevblogaday.com/feed/ qui ne passe pas chez moi non plus https://github.com/tontof/kriss_feed/issues/57 mais qui est valide (j'ai libxml 2.7.8)

jcb commented 11 years ago

Je viens de tester (même conf, index.php mis à jour) et je me fais insulter : "The feed you are trying to add already exists or is wrong. Check your feed or try again later."

dacrovinunghi commented 11 years ago

ok chez en forcant l'url (redirection avec un index.php modifié) libxml Version 2.7.6 redirection curl off

tontof commented 11 years ago

Arrêtez tout j'ai trouvé :-) Je patche dans la matinée

C'est juste une simple question de données compressées qui arrivent et qu'il faut décompresser avant :-) d'où le Warning: DOMDocument::load(): Start tag expected, '<' not found

tontof commented 11 years ago

Est-ce que des gens qui avaient des problèmes avec Cyrille, enfin je veux dire avec son flux RSS : http://www.cyrille-borne.com/index.php?feed/atom http://www.cyrille-borne.com/index.php?feed/rss2 Peuvent me confirmer qu'avec ce nouvel https://github.com/tontof/kriss_feed/raw/master/src/index.php ça marche ? Merci

memiks commented 11 years ago

Je viens de tester et ça marche ;)

http://rss.memiks.fr/

tontof commented 11 years ago

Et ça marchait pas avant ?

memiks commented 11 years ago

Non avant ça ne marchait pas j'avais le bug du flux déjà présent ou incorrect.

ghost commented 11 years ago

félicitations à tous, désormais vous aurez la garantie d'avoir du Troll. Et sinon est ce que quelqu'un a fait l'essai avec un autre dotclear. Sinon pour info mon blog a toujours fonctionné chez moi mais il s'agit d'un import de l'opml.

tontof commented 11 years ago

Victoire !!!!

En fait le problème dépend de plusieurs critères. Le serveur sur lequel est hébergé KrISS feed et ton hébergeur (@cborne). Si les 2 sont bien configurés et qu'ils se mettent d'accord, les données sont transférées en étant zippées et sont bien décompressées, sinon je pense que ton serveur envoie en zippé par défaut alors que le serveur ne l'avait pas précisé et là soit le serveur dézippait quand même soit pas et ça plantait.

J'espère que c'est bien ça. J'ai hâte d'avoir confirmation de @sebsauvage mais j'ai bon espoir grâce à @memiks

Je ne peux pas tester avec un autre dotclear, ça marchait déjà comme il faut pour moi avant

ghost commented 11 years ago

Mon hébergeur c'est o2switch le meilleur hébergeur Auvergnat du monde.

Il faudrait faire le test avec le flux de la collègue qui est chez o2switch aussi : http://www.readingintherain.com/feed/

mtexier commented 11 years ago

Salut,

J'ai re-testé http://www.altdevblogaday.com/feed/ (bug #57) et ça marche avec la version que tu viens de mettre juste au dessus. Super réactivité! Merci!

tontof commented 11 years ago

Bon je ferme alors et on avisera si y a d'autres problèmes :-)

Edit : j'ai lu trop vite, je croyais que tu parlais du flux de Cyrille... J'attends encore une confirmation pour fermer l'issue

mtexier commented 11 years ago

Celui la http://www.cyrille-borne.com/index.php?feed/rss2 marche chez moi avec ta dernière version ;)

tontof commented 11 years ago

Oui mais est-ce qu'il ne marchait pas avant ?

sebsauvage commented 11 years ago

Arg ben désolé mais le flux /rss2 de Cyrille ne passe toujours pas avec la version que j'ai prise il y a quelques minutes là: https://github.com/tontof/kriss_feed/raw/master/src/index.php

Edit: Le flux ATOM ne passe pas non plus.

2013-03-20_205919

Désolé je ne peux pas faire de Wireshark sur mon mutualisé.

ghost commented 11 years ago

@sebsauvage est ce que tu peux faire un essai avec ce site qui est aussi sur o2switch voir si ça vient de l'hébergeur : http://www.readingintherain.com/feed/

dacrovinunghi commented 11 years ago

tof j'ai vu ta redirection curl redisdead

avec la sale redirection que je fais dans loadxml, le flux passe

code // fix url 301 302 //** $res=get_headers($xmlUrl); $redi= $res[0]; echo $redi; print_r ($res); $i=5; while ( $i>0 && (substr($redi,0,11) ==="HTTP/1.1 30")) { foreach( $res as $value ) { if ( substr( strtolower($value), 0, 9 ) == "location:" ) {
$xmlUrl= substr( $value, 10 ); echo "location ",$i ," : ",$xmlUrl,"\n"; } } echo "relocation ",$i ," : ",$xmlUrl,"\n"; $res=get_headers($xmlUrl); $redi= $res[0]; //echo "code redi",$redi; $i--;

}

return($xmlUrl); }

tontof commented 11 years ago

@sebsauvage tu as essayé plusieurs fois ? J'étais en train de te faire un script réduit et j'ai eu ça comme résultat :

$ php5 loadXML.php 
PHP Warning:  DOMDocument::loadXML(): Empty string supplied as input in /var/www/git/tmp/loadXML.php on line 65
PHP Stack trace:
PHP   1. {main}() /var/www/git/tmp/loadXML.php:0
PHP   2. DOMDocument::loadXML() /var/www/git/tmp/loadXML.php:65
length: 0
start: 
$ php5 loadXML.php 
length: 63891
start: <?xml vers

Du coup sans rien changer ça a marché la 2e fois

Edit : @dacrovinunghi je ne crois qu'il s'agisse d'un problème de redirection ici.

sebsauvage commented 11 years ago

@cborne: J'ai exactement la même erreur avec http://www.readingintherain.com/feed/

tontof commented 11 years ago

@sebsauvage je viens de t'envoyer un code par mail avec juste la fonction qui télécharge. Je vois vraiment pas ce qui bloque, tu as une idée ?

sebsauvage commented 11 years ago

@tontof: Strict Standards: Non-static method DOMDocument::loadXML() should not be called statically in /home/sebsauva/www/test/loadXML.php on line 65

Warning: DOMDocument::loadXML() [domdocument.loadxml]: Empty string supplied as input in /home/sebsauva/www/test/loadXML.php on line 65 length: 0 start:

dacrovinunghi commented 11 years ago

sur ces 2 configs (avec salle redirection...) ca passe libxml Version 2.7.6 et libxml Version 2.8.0 http://www.readingintherain.com/feed/ aussi

sebsauvage commented 11 years ago

D'après la doc, un appel statique à loadXML() génère une erreur (ce qui est mon cas): http://php.net/manual/fr/domdocument.loadxml.php

Donc: $document = new DOMDocument(); $document->loadXML($output);

mais ça ne corrige que la première des deux erreurs que j'ai.

Je présume que c'est l'appel à curl qui pose problème. Je regarde.

sebsauvage commented 11 years ago

Le $output = curl_exec_follow($ch); renvoie false.

dacrovinunghi commented 11 years ago

LA vrai erreur c'est Empty string car pas de xml en entrée donc url incorrecte probable. pour le débug afficher le xml