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

Add a 'logout' protection #167

Closed
Nono-m0le opened this issue Apr 8, 2013 · 20 comments
Closed

Add a 'logout' protection #167

Nono-m0le opened this issue Apr 8, 2013 · 20 comments

Comments

@Nono-m0le
Copy link
Contributor

[Désolé, j'ai la flemme d’écrire en anglais pour aujourd'hui...]

Parfois, lorsque je laisse mon kriss pdt 2/3 heures sans action et que j'y retourne, je peux toujours l'utiliser avec les raccourcies clavier.
C'est qu'une fois que je refresh que je me rend compte que je suis déconnecté .... et que les feeds que j'avais marqué en lu ne l'était pas.

Ca serait pas mal d'avoir une verif !

@tontof
Copy link
Owner

tontof commented Apr 8, 2013

Héhé, pas de problème pour l'anglais.
J'ai eu un comportement similaire, mais comme je suis en unread, je me rends vite compte que je ne suis plus connecté parce que les éléments réapparaissent assez rapidement :-)
Je vais essayer de penser à ça ^^

@tontof
Copy link
Owner

tontof commented Apr 9, 2013

Une façon de voir si tu es connecté, mais normalement ton nombre de Unread dans le titre ne change pas si ?
En fait, j'ai dû mal à voir comment intégrer ça facilement pour laisser la possibilité de l'utiliser en protected.

@Nono-m0le
Copy link
Contributor Author

Je viens de tester ... il diminue aussi !
Test : j'appuie sur "m" (il est caché, et je passe de 65 à 64)
Je clic sur "All Feed", et pouf, je dois me reloguer
Une fois reloguer : j'ai mes 65 flux, et mon flux marqué en read (normalement) qui est a nouveau là

@tontof
Copy link
Owner

tontof commented Apr 9, 2013

Tu es en quel mode ? (public, protected, private ?)

Sur la démo : http://tontof.net/feed/
T'es d'accord que si tu appuyes sur 'm' ça ne change le nombre de unread dans le titre ?

Moi si j'essaye en mode private :

  • je me connecte sur la page 1
  • je me déconnecte sur une autre page (page 2)
  • si j'appuye sur 'm' sur la page 1 le nombre de unread ne change pas

@Nono-m0le
Copy link
Contributor Author

Je suis en Private

Sur ton feed : Quand j'appuie sur "m", All Feed baisse d'1 et le répertoire du flux baisse aussi de 1
Par contre, le TITRE de la fenetre (balise <title></title>, lui, oui, ne change pas => mais je le vois jamais ce titre ^^)

@tontof
Copy link
Owner

tontof commented Apr 9, 2013

Ah ben oui, mais je parlais de ce titre là moi :-)
J'essaye de voir comment je peux ajouter ça.

@Nono-m0le
Copy link
Contributor Author

sebsauvage/Shaarli#49 (comment)
Tu avais déjà lu/vu ?

@tontof
Copy link
Owner

tontof commented Apr 11, 2013

Oui, t'as essayé pour voir si ça marchait chez toi ?

@Nono-m0le
Copy link
Contributor Author

absolument pas ! J'y pensais depuis un moment en fait (quand j'ai vu l'article shaarli passer...), je me suis dit que t’étais déjà dessus :p
Mais comme ca m'arrive entre 20 et 30fois par jour, je dois être plus pressé que toi :)
J'aurais quoi à faire ?

@tontof
Copy link
Owner

tontof commented Apr 11, 2013

En essayant d'ajouter ça dans le fichier index.php
ini_set("session.gc_maxlifetime", '86400');
ini_set('session.save_path', $_SERVER['DOCUMENT_ROOT'].'/sessions');

Je suis pas du tout pressé, je me limite à lire mes flux le matin et le soir en ce moment :-)
Donc j'ai pas de problème de déconnexion :-p

@tontof
Copy link
Owner

tontof commented Apr 16, 2013

J'ai ajouté un alert en javascript quand il y a eu une déconnexion :-)
7547e06

@tontof tontof closed this as completed Apr 16, 2013
@Nono-m0le
Copy link
Contributor Author

classe, avec les 2 ini_set (et la création du dossier qui va bien, sinon wrong token !) : plus de déco :D Ca marche au poil, je fais cette petite update en rajoutant les params :p (ca serait pas mal d'avoir ca en options, pour forcer le non logout pour ceux qui peuvent).

@tontof
Copy link
Owner

tontof commented Apr 16, 2013

Arf je vais voir si je peux ajouter ça sans que ça pose trop de problème.
En tout cas c'est cool :-)

@Nodd
Copy link

Nodd commented Apr 26, 2013

👍
Ça m'intéresse d'avoir cette option, j'ai testé et c'est quand même plus agréable quand je ne dois pas me reconnecter toutes les quelques heures...

@Nono-m0le
Copy link
Contributor Author

Alelouia ! :)

@tontof
Copy link
Owner

tontof commented Apr 26, 2013

@Nodd Tu as essayé et ça marche ?
J'ai ajouté à #97, maisje ne sais toujours pas comment je vais pouvoir intégrer ça

@Nono-m0le
Copy link
Contributor Author

Juste les deux lignes si l'option est activée, et la verif/création du dossier session :)

@Nodd
Copy link

Nodd commented Apr 28, 2013

Oui c'est ça, j'ai essayé et j'ai pas eu de déco en 3-4 jours, alors qu'avant c'était au moins 3 fois par jour.
Ça ne me parait pas compliqué d'ajouter une option et les deux lignes (+ dossier session) comme dit Nono, c'est quoi qui te dérange en particulier ?

@tontof
Copy link
Owner

tontof commented Apr 29, 2013

Je ne voulais pas ajouter ce comportement par défaut parce qu'il s'agit d'un problème spécifique lié à la config serveur et je pense qu'il est plus intéressant de le faire avec un plugin. Du coup ça attendra un peu. En tout cas c'est cool que ça marche en faisant ça.

@Sbgodin
Copy link

Sbgodin commented Dec 6, 2013

FR: J'ai fait un "Stay signed in" qui est parmanent, voir #155.
EN: I made a "Stay signed in" which is permanent, see #155.

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

4 participants