search

FranckRJ / RespawnIRC-Android

Version android de RespawnIRC.
https://pijon.fr/RespawnIRC-Android/
zlib License
13 stars 6 forks source link

Mise à jour cookies dans la webview #17

Closed CAP-ONE closed 3 years ago

CAP-ONE commented 3 years ago

J'ai remarqué que la popup apparait toujours brièvement si on fait une mise à jour ou qu'on laisse intact les data de l'application.

A mon avis ça doit venir du système de Webview qui à des infos en cache de façon permanente, doit y avoir un moyen de les clear une fois pour toute dès le lancement de l'application.

FranckRJ commented 3 years ago

C'est parce que j'ai changé la manière dont la popup est gérée, au lieu de pré-remplir la webview avec des cookies pour faire croire que le bouton a déjà été cliqué, j'injecte du JavaScript pour cliquer sur le bouton.

Parce qu'en fait je me rends compte que set des cookies par défaut ça marche jamais très bien.

Le lun. 12 avr. 2021 à 13:49, Rodney V. @.***> a écrit :

J'ai remarqué que la popup apparait toujours brièvement si on fait une mise à jour ou qu'on laisse intact les data de l'application.

A mon avis ça doit venir du système de Webview qui à des infos en cache de façon permanente, doit y avoir un moyen de les clear une fois pour toute dès le lancement de l'application.

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/FranckRJ/RespawnIRC-Android/pull/17#issuecomment-817744009, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABX5ETYEJ2Z76PW65TK77P3TILM3HANCNFSM42XCJGYQ .

CAP-ONE commented 3 years ago

J'avais pas vu le commit lié en effet, c'est plus simple de gérer ainsi oui mais sachant que la page à besoin d'être chargée pour executer le JS, et que du coup les cookies sont clear à chaque création de la Webview, elle apparaitra toujours brièvement, et peut vite devenir irritant à force pour l'utilisateur.

Je suis pour cela partisan du chargement des cookies lors de la création de la Webview afin que ça demeure invisible pour l'utilisateur

FranckRJ commented 3 years ago

Si tu trouves une solution qui fonctionne durablement je prends.

Le lun. 12 avr. 2021 à 14:41, Rodney V. @.***> a écrit :

J'avais pas vu le commit lié en effet, c'est plus simple de gérer ainsi oui mais sachant que la page à besoin d'être chargée pour executer le JS, et que du coup les cookies sont clear à chaque création de la Webview, elle apparaitra toujours brièvement, et peut vite devenir irritant à force pour l'utilisateur.

Je suis pour cela partisan du chargement des cookies lors de la création de la Webview afin que ça demeure invisible pour l'utilisateur

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/FranckRJ/RespawnIRC-Android/pull/17#issuecomment-817777749, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABX5ETZJWCEPS4FCROINHILTILTAXANCNFSM42XCJGYQ .

CAP-ONE commented 3 years ago

Si JVC change encore leur système il faudra également apporter des modifs dans les deux cas. C'est pas un soucis en soit dans la mesure où on devra juste mettre à jour l'application.

J'avais imaginé un système où on pourrait insérer nous même des cookies depuis une option dans la webview, mais à mon avis trop contraignant pour les utilisateurs non aguerris.

FranckRJ commented 3 years ago

Le truc c'est que la solution des cookies casse sans que JVC change quoi que ce soit, les valeurs deviennent obsolètes et ne sont plus valides. Les valeurs doivent être générées dynamiquement (ou récupérées via une requête), mais ça prend du temps de voir comment faire exactement et j'ai la flemme.

Le lun. 12 avr. 2021 à 14:57, Rodney V. @.***> a écrit :

Si JVC change encore leur système il faudra également apporter des modifs dans les deux cas. C'est pas un soucis en soit dans la mesure où on devra juste mettre à jour l'application.

J'avais imaginé un système où on pourrait insérer nous même des cookies depuis une option dans la webview, mais à mon avis trop contraignant pour les utilisateurs non aguerris.

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/FranckRJ/RespawnIRC-Android/pull/17#issuecomment-817788509, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABX5ET2CQSULSKDTUNZFEO3TILU4JANCNFSM42XCJGYQ .

FranckRJ commented 3 years ago

Quand j'avais testé ta PR hier soir ça fonctionnait pas chez moi, je voyais toujours la popup. Je sais pas exactement à quoi c'est dû.

Le lun. 12 avr. 2021 à 15:03, Franck W. @.***> a écrit :

Le truc c'est que la solution des cookies casse sans que JVC change quoi que ce soit, les valeurs deviennent obsolètes et ne sont plus valides. Les valeurs doivent être générées dynamiquement (ou récupérées via une requête), mais ça prend du temps de voir comment faire exactement et j'ai la flemme.

Le lun. 12 avr. 2021 à 14:57, Rodney V. @.***> a écrit :

Si JVC change encore leur système il faudra également apporter des modifs dans les deux cas. C'est pas un soucis en soit dans la mesure où on devra juste mettre à jour l'application.

J'avais imaginé un système où on pourrait insérer nous même des cookies depuis une option dans la webview, mais à mon avis trop contraignant pour les utilisateurs non aguerris.

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/FranckRJ/RespawnIRC-Android/pull/17#issuecomment-817788509, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABX5ET2CQSULSKDTUNZFEO3TILU4JANCNFSM42XCJGYQ .

CAP-ONE commented 3 years ago

Dans un comportement normal le navigateur enregistre la demande une seule fois et la popup n'apparait pas tant que les cookies existent.

J'ai pas vraiment testé dans le cas d'une mise à jour de l'application, c'est pour cela que j'ai avancé l'hypothèse du cache persistant. Je regarderai en détail quand j'aurai le temps.