Projet

Général

Profil

Bug #8806

page d'accuel du portail agent vide

Ajouté par Serghei Mihai il y a plus de 8 ans. Mis à jour il y a environ 7 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
28 octobre 2015
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Club:

Description

Sous IE 11 la page d'accueil du portail agent est vide.
Le debuggeur affiche une erreur "Access denied" au localStorage


Fichiers

Historique

#2

Mis à jour par Frédéric Péters il y a plus de 8 ans

Il est noté dans un mail que ça concerne aussi firefox, ne perdons pas d'infos dans les tickets.

Serghei, tu regardes aux liens postés par Benjamin ?

#3

Mis à jour par Serghei Mihai il y a plus de 8 ans

  • Statut changé de Nouveau à En cours
  • Assigné à mis à Serghei Mihai

La plus simple pour contourner l'erreur sous IE est de desactiver le mode protegé: "Internet Options" -> onglet "Security", decocher "Enable Protected Mode".
Avec le dernier Firefox pour windows(v41.0.2) le problème n'est pas présent.

Pierre a demandé à Lydie quelle version elle avait.

#4

Mis à jour par Pierre Cros il y a plus de 8 ans

Le mercredi 28 octobre 2015 à 14:45 +0100, a écrit :

Pierre a demandé à Lydie quelle version elle avait.

Et Lydie dit (waahhhaahahaha) "ma version de Firefox est 38.3.0." 

#5

Mis à jour par Serghei Mihai il y a plus de 8 ans

Testé avec la version 38.3.0 sur Windows et aucun problème.
Pareil sur la version 38.3.0 sur ma Sid.

On peut tenter de lui demander de lancer une session privée(Ctrl+Shit+P sous Win)?

#6

Mis à jour par Frédéric Péters il y a plus de 8 ans

Serghei, la confirmation de IE, c'est la configuration par défaut ?

#7

Mis à jour par Serghei Mihai il y a plus de 8 ans

Le "Protected mode" est activé par défaut.

#8

Mis à jour par Benjamin Dauvergne il y a plus de 8 ans

Mail envoyé sur travailleurs-prive:

J'ai trouvé cela sur les politiques de sécurité possibles au niveau de Firefox et concernant
localstorage:

http://superuser.com/questions/629525/how-to-control-websites-use-of-localstorage-in-firefox

Ce sont les politiques sur les cookies (pas de cookie ou session seulement) qui s'étendent au
localstorage.

Peut-être faudrait-il plutôt utiliser sessionStorage pour l'usage qu'on en fait, sachant que nos
cookies de session sont aussi liés à la session.

#9

Mis à jour par Benjamin Dauvergne il y a plus de 8 ans

Si on veut quand même utiliser localStorage, coté firefox on doit pouvoir chopper une SecurityError au niveau du JS pour signaler à l'utilisateur que l'expérience risque d'être dégradé s'il n'autorise pas les cookies longue durée pour ce site1.

[1]: http://meyerweb.com/eric/thoughts/2012/04/25/firefox-failing-localstorage/

#10

Mis à jour par Frédéric Péters il y a plus de 8 ans

En fait on utilise sessionStorage pour le contenu du menu publik,

    window.sessionStorage.hobo_environment = JSON.stringify(COMBO_KNOWN_SERVICES);

Là où on utilise localStorage, c'est pour mémoriser la préférence "menu publik ouvert ou fermé" de l'usager; ce qu'on pourrait en fait considérer comme plutôt accessoire.

#12

Mis à jour par Benjamin Dauvergne il y a plus de 8 ans

  • Assigné à mis à Serghei Mihai

Frédéric Péters a écrit :

En fait on utilise sessionStorage pour le contenu du menu publik,

[...]

Là où on utilise localStorage, c'est pour mémoriser la préférence "menu publik ouvert ou fermé" de l'usager; ce qu'on pourrait en fait considérer comme plutôt accessoire.

Possible qu'un crash au niveau du code pour le sidepage_status crash le reste ? Si c'est le cas il faudrait faire un modification locale pour utiliser sessionStorage et voir si ça débloque Lydie.

Dans l'absolu je dirai qu'on pourrait avoir une approche "amélioration progressive", on tente localStorage, sinon sessionStorage, sinon cookie de session sinon rien.

Si sessionStorage n'est pas accessible pour le hobo_environment on ne crash pas on ouvre juste une popup d'alerte disant que le site est incompatible avec le navigateur à cause de l'absence de sessionStorage.

#13

Mis à jour par Benjamin Dauvergne il y a plus de 8 ans

Frédéric Péters a écrit :

Quelque chose du genre.

Ack, on teste en prod (ou recette) montpellier ?

#14

Mis à jour par Benjamin Dauvergne il y a plus de 8 ans

Je me dis que si sessionStorage risque de péter une erreur sur une simple référence on pourrait aussi ajouter "storage = {}", après un try/catch sur sessionStorage et en dernier recourt (ça ne retient rien mais ça me crashera pas dans une même page); pour hobo_environment sur IE il vaut mieux utiliser un "alert()" j'ai peur sinon que les appels JSONP à chaque chargement de page ne soient trop nombreux.

#15

Mis à jour par Frédéric Péters il y a plus de 8 ans

Et patch côté wcs, je vais appliquer les deux sur la prod de Montpellier, c'est le plus facile pour Lydie et cie.

#16

Mis à jour par Frédéric Péters il y a plus de 8 ans

Et en fait, on va plutôt tester en interne; Serghei, fait chauffer ta VM windows.

#17

Mis à jour par Serghei Mihai il y a plus de 8 ans

Ok. Tu appliques les patchs en dev?

#18

Mis à jour par Frédéric Péters il y a plus de 8 ans

testé et validé avec IE en dev; je copie sur la prod pour validation par Montpellier.

#19

Mis à jour par Frédéric Péters il y a plus de 8 ans

C'est mis en place sur la prod; Serghei tu pourras vérifier par toi-même sur la prod et si c'est ok leur envoyer un mail pour demander de tester (après avoir généreusement rechargé la page à coup de F5) ?

#20

Mis à jour par Serghei Mihai il y a plus de 8 ans

Sur la prod j'ai toujours l'erreur, même après avoir vidé le cache du navigo.
Le patch a bien été appliqué?

#21

Mis à jour par Frédéric Péters il y a plus de 8 ans

côté wcs mais côté combo il manquait un collectstatic, fait maintenant. (et pareil pour hobo/passerelle/authentic).

#22

Mis à jour par Serghei Mihai il y a plus de 8 ans

Validé avec Montpel.
Ack pour le patch.

#23

Mis à jour par Frédéric Péters il y a plus de 8 ans

  • Statut changé de En cours à Résolu (à déployer)
commit a9219eaabaf4c85d45f9138221797c9e26ea32cc
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Fri Nov 6 09:21:20 2015 +0100

    use sessionStorage if localStorage is not available (#8806)
#24

Mis à jour par Serghei Mihai il y a environ 7 ans

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF