Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Quelques problèmes avec le flux RSS #48

Closed
arthurlutz opened this issue Jul 30, 2014 · 5 comments
Closed

Quelques problèmes avec le flux RSS #48

arthurlutz opened this issue Jul 30, 2014 · 5 comments
Labels

Comments

@arthurlutz
Copy link

J'ai une installation minigalnano où j'aimerais intégrer le flux RSS dans un lecteur d'images qui sait lire du mediarss. Ca ne marche pas (mais bon, c'est peut-être un bug de l'application en question), du coup j'ai regardé la validité du flux.

Selon http://feedvalidator.org/ j'avais une erreur sur un item où une image contient des espaces dans le nom de fichier de la photo. Il faut probablement échapper le nom de fichier.

Autre remarque : il fait des warning sur "item should contain a guid element" http://feedvalidator.org/docs/warning/MissingGuid.html et "Missing atom:link with rel="self"" http://feedvalidator.org/docs/warning/MissingAtomSelfLink.html

Autre truc : le mimetype ne semble pas être xml mais "text/html".

@nodiscc
Copy link
Contributor

nodiscc commented Aug 31, 2014

J'ai quelques warnings identiques:

This feed is valid, but interoperability with the widest range of feed readers could be improved by implementing the following recommendations

  • Feeds should not be served with the "text/html" media type (à fixer dans rss.php)
  • item should contain a guid element (25 occurrences)
  • Missing atom:link with rel="self" (il faut une balise <atom:link href="$gallery_link/rss.php" rel="self" type="application/rss+xml" /> à l'intérieur des balises <channel>)

Je n'arrive pas à parser le flux RSS avec moonmoon (un aggrégateur/planet en PHP) pour une raison inconnue. Peut être que fixer ces quelques warnings arrangerait les choses?

@tmos
Copy link
Collaborator

tmos commented Nov 9, 2014

N'hésites pas à faire la pull de ton code pour le flux RSS @nodiscc :-)

@tmos tmos added the bug label Nov 11, 2014
@Lucas-C
Copy link
Contributor

Lucas-C commented Dec 5, 2014

J'ai également un souci avec le flux RSS: tous les images sont agglutinés en un seul item.
Exemple: https://chezsoi.org/lucas/wwcb/rss.php
Est-ce normal ? Je pensais que les items seraient ordonnés par date de modification / création. Et je n'ai trouvé aucune documentation là-dessus.

EDIT: en rendant le répertoire writable (chmod o+w .) cela a permis à trois fichiers db_* d'être créés. Cependant le problème persiste : si j'ajoute un fichier, il n'apparaît pas en tête du flux.
Est-ce que quelqu'un de familier avec rss.php sait dans quel ordre les images sont parcourues ?
https://github.com/sebsauvage/MinigalNano/blob/master/rss.php#L160

EDIT: ok, j'ai parlé trop vite. Tout fonctionne ok.
Je voudrais juste suggérer de documenter tout ça quelque part.
Est-ce suffisant que le dossier soit writable à la première exécution, pour que les fichiers db_* soit créés ? J'ai l'impression que le fichier rss.locker doit être créé à chaque MAJ du flux, mais je préfererais ne pas laisser le dossier writable par sécurité.

@nodiscc
Copy link
Contributor

nodiscc commented Dec 22, 2014

J'ai ajouté les changements nécessaires dans #87

Je laisse tomber le fix pour Missing atom:link with rel="self", c'est optionnel et la spec RSS est respectée même sans.

J'ai toujours un warning, relatif à des espaces non échappés dans un nom de fichier: Invalid character in a URI: https://my.server.net/images/photos/Archive/audiomz/1994.04.24 Notator SL.jpg

Lucas-C pushed a commit to Lucas-C/MinigalNano that referenced this issue Feb 21, 2015
Lucas-C pushed a commit to Lucas-C/MinigalNano that referenced this issue Feb 21, 2015
tmos pushed a commit that referenced this issue Jun 29, 2015
@tmos
Copy link
Collaborator

tmos commented Jun 29, 2015

I fixed the invalid character with a simple str replace. Does it seems okay on your side ?

@tmos tmos closed this as completed Jul 1, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants