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

Ne pas rafraîchir les flux des utilisateurs non logués depuis x jours/semaines/mois #681

Closed
ldidry opened this issue Oct 26, 2014 · 9 comments
Assignees
Labels
Framasoft 🐇 UI 🎨 User Interfaces Work in progress 🚧 Wait before merging
Milestone

Comments

@ldidry
Copy link
Contributor

ldidry commented Oct 26, 2014

Les utilisateurs n'utilisant pas le service depuis un certain temps ne devraient pas avoir leur flux rafraîchis. Cela utilise des ressources inutilement.

Dans l'interface d'admin, un champ number fera l'affaire :

= 0 => openbar

!= 0 => pas de rafraîchissement si pas logué depuis X jours

@Alwaysin
Copy link
Contributor

Toutes tes propositions se rapprochent beaucoup d'un service qui serait proposé "en masse" à des utilisateurs, du style wallabag qui est proposé par Framasoft via Framabag. Je ne sais pas si @marienfressinaud a déjà réfléchi à l'idée de proposer un tel service ?

@marienfressinaud
Copy link
Member

@Alwaysin > si si, j'y pense :p

@ldidry > je répondrai aux tickets plus tard ;)

@ldidry
Copy link
Contributor Author

ldidry commented Oct 26, 2014

@Alwaysin Oui, absolument, mes propositions sont à mettre en relation avec Framanews. Nous utilisons une version légèrement modifiée de Tiny Tiny RSS, mais les mises à jour sont assez complexes, et interagir avec le dév principal est… comment dire… Il faut aller voir le forum de ttrss pour appréhender la personnalité du personnage (très compétent au demeurant).

@marienfressinaud No pb. Comme tu as pu le voir, j'ai mis en place un Freshrss sur framanews.org, pour tester. J'aime beaucoup, mais du coup, je teste la scalabilité (appel à inscription) et je crée plein de tickets pour implémenter ce dont on a besoin sur Framanews :-)

@Alkarex
Copy link
Member

Alkarex commented Oct 26, 2014

Cela me semble facilement réalisable. Comme pistes de discussion, je suggère :

  1. De mettre rapidement une option globale commune à l'ensemble des utilisateurs.
  2. Plus tard, avec le futur système d'extension, fournir des événements qui permettront à une extension de décider des choses comme autoriser le rafraîchissement, autoriser l'ajout de nouveaux flux, autoriser l'ajout de nouvelle catégories, etc.

Alkarex added a commit that referenced this issue Oct 28, 2014
Alkarex added a commit that referenced this issue Oct 28, 2014
Before the cache system was not used for feeds with errors, which was
problematic especially if several users have this feed.
Furthermore, there was no protection against repetitive refresh.

Bonus: slightly better performance by avoiding some superfluous
file_exists().

Warning: needs a bit of testing
#681
Alkarex added a commit that referenced this issue Oct 28, 2014
@Alkarex
Copy link
Member

Alkarex commented Oct 28, 2014

Voilà qui est fait pour (1) 9f97f7d avec une nouvelle option dans ./data/config.php.

'limits' =>
  array (
    'max_inactivity' => -1, //Time in seconds after which a user who has not used the account is considered inactive (no auto-refresh of feeds).
  ),

Une valeur négative équivaut à la valeur par défaut (2147483647 secondes).
Ce n'est pas encore testé en profondeur.

@marienfressinaud
Copy link
Member

À noter que je souhaite ajouter à terme un système pour lister les 5 dernières connexions de l'utilisateur et à partir de quelle adresse IP (lié à #501).

Avant ça j'aimerais déplacer les fichiers utilisateurs dans un répertoire dédié à chaque utilisateur : configuration et base de données SQLite pour le moment, et ensuite historique de connexion ainsi que, éventuellement un deuxième fichier de configuration pour mobile (#264).

@Alkarex
Copy link
Member

Alkarex commented Oct 29, 2014

Bonne idée

@Alkarex
Copy link
Member

Alkarex commented May 16, 2015

@ldidry Avez-vous eu l'occasion d'essayer l'option max_inactivity introduite il y a quelques mois en réponse à votre demande ?

@Alkarex Alkarex added this to the 1.1.2 milestone May 16, 2015
@Alkarex Alkarex added Work in progress 🚧 Wait before merging UI 🎨 User Interfaces and removed Extension 🔌 labels May 16, 2015
@Alkarex
Copy link
Member

Alkarex commented May 16, 2015

A priori, il ne manque qu'une interface Web pour modifier les limites plus facilement. Voir #680 pour la suite.

@Alkarex Alkarex closed this as completed May 16, 2015
@Alkarex Alkarex self-assigned this May 23, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Framasoft 🐇 UI 🎨 User Interfaces Work in progress 🚧 Wait before merging
Projects
None yet
Development

No branches or pull requests

4 participants