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

[doc] taille de l'OPML à l'import #228

Closed
tetraf opened this issue Oct 30, 2013 · 12 comments
Closed

[doc] taille de l'OPML à l'import #228

tetraf opened this issue Oct 30, 2013 · 12 comments
Assignees
Milestone

Comments

@tetraf
Copy link

tetraf commented Oct 30, 2013

Il pourrait être intéressant de mentionner dans la doc ou dans l'interface que l'import d'un OPML peut être long et dépasser max_execution_time de PHP.

On peut le régler dans la configuration d'Apache par exemple, au moins le temps de faire l'import, ainsi :

php_admin_value max_execution_time "300"

Ce qui évite de rencontrer ce genre d'erreur :

PHP Fatal error:  Maximum execution time of 30 seconds exceeded in /srv/http/freshrss/lib/SimplePie/SimplePie/Sanitize.php on line 457
@Alkarex
Copy link
Member

Alkarex commented Nov 4, 2013

Je pense qu'il faudra aussi revoir le mécanisme d'import pour le faire en plusieurs fois et éviter ce problème.

@mathieumd
Copy link

Il semble aussi que « simplement » rejouer l'importation plusieurs fois soit bien géré et ne fasse pas de doublon.

@Alkarex
Copy link
Member

Alkarex commented Dec 1, 2013

Je regarde un peu le problème. Avec quelle version de FreshRSS est-ce que ce problème de durée d'exécution a été rencontré, et à quel moment (importation OPML ou raffraîchissement par exemple) ?

Alkarex added a commit that referenced this issue Dec 1, 2013
Ajout de quelques set_time_limit pour
#228
@mathieumd
Copy link

J'ai eu le timeout avec FreshRSS v0.6.1 et un fichier OPML de 20 701 octets. J'ai l'impression que ça timeout pendant l'importation même, car il manquait quelques flux.

@tetraf
Copy link
Author

tetraf commented Dec 1, 2013

Moi c'était avec une version 0.5.0, à l'import de l'OPML (68K) je voyais une page blanche dans mon navigateur et, dans les logs de mon serveur Web, il y avait l'erreur susmentionnée.

@ghost ghost assigned Alkarex Dec 1, 2013
@Alkarex
Copy link
Member

Alkarex commented Dec 1, 2013

Ok, merci.
J'ai ajouté un set_time_limit qui devrait déjà fonctionner dans beaucoup de cas.
Néanmoins, il faudrait retravailler l'import des favicons qui sont actuellement tous téléchargés lors de la même requête initiale après l'import de l'OPML. Je tâcherai de faire cela dans les jours à venir.
Voir aussi #290

Alkarex added a commit that referenced this issue Dec 1, 2013
Nouvelle méthode pour afficher les favicons à la demande et en
parallèle.
Déplacement du dossier des favicons sous /data/favicons/
Devrait permettre de fermer le problème d'import OPML
#228
Voir aussi #290
Alkarex added a commit that referenced this issue Dec 1, 2013
Alkarex added a commit that referenced this issue Dec 1, 2013
Clôture #228
Nous perdons le champ description des flux, mais celui-ci pourrait par
exemple être récupéré au premier rafraîchissement du flux si nécessaire
@Alkarex
Copy link
Member

Alkarex commented Dec 1, 2013

L'import OPML est maintenant instantané :-)

@Alkarex Alkarex closed this as completed Dec 1, 2013
Alkarex added a commit that referenced this issue Dec 2, 2013
Charge et sauve la description des flux.
Redirige vers la page d'accueil après une importation OPML (maintenant
rapide, et pour mieux permettre le rafraîchissement des flux avec moins
de risques que l'utilisateur quitte la page)

Suite de #228
Alkarex referenced this issue Dec 2, 2013
Plus correction bug checkUrl()
@Alkarex
Copy link
Member

Alkarex commented Dec 2, 2013

Je pense avoir fini de retravailler l'import OPML.
Si ça vous dit de tester la branche /dev (attention, pas compatible v0.6 pour l'instant, donc il faut faire une installation fraiche), n'hésitez pas à donner des retours.

@mathieumd
Copy link

Je viens d'installer dev pour tester #322, et quand je clique sur « Importer / exporter (OPML) » j'ai une page blanche, et les logs d'erreurs de Nginx :

FastCGI sent in stderr:
"PHP message: PHP Fatal error:  Class 'HelperFeed' not found in
/[...]/[...]/[...]/FreshRSS-dev/app/Models/CategoryDAO.php on line 225"
while reading response header from upstream

@Alkarex
Copy link
Member

Alkarex commented Dec 19, 2013

Merci, je vais essayer de regarder cela ce soir.
Il y a eu pas mal de changements depuis mon dernier test d'installation.

@Alkarex
Copy link
Member

Alkarex commented Dec 19, 2013

Voilà @mathieumd, le problème devrait être réglé :-)

@mathieumd
Copy link

En effet :-) 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants