Problème de multi-session

Version imprimable

01/03/2011, 13h49
jgfa9

Problème de multi-session
Bonjour,
Je développe une appli web et j'ai un problème de session : quand un utilisateur se connecte, je mets un objet "user" (de type User : nom, userid, ...) en session (fichier SignonAction.java):
Code:

1 2 3 ... request.getSession().setAttribute("USER", user); ...
Mes classes action héritent toutes d'une classe BaseAction.java qui créé une variable de type User :

Code:

User user = (User) request.getSession().getAttribute("USER");

instancié dans la fonction

Code:

execute(ActionMapping mapping, ActionForm form, HttpServletRequest request, HttpServletResponse response)

Ainsi, je peux théoriquement accéder à l'objet "user" dans toutes mes classes.

Or, il se trouve qu'un utilisateur qui lance un calcul assez long voit ce calcul s'arrêter brutalement ; le fichier de trace indique une erreur de type java.lang.NullPointerException
et la ligne en cause est la suivante :

Code:

un_autre_objet.setUserid(user.getUserid());

Or il se trouve que cela arrive systématiquement quand une autre personne se connecte, quelle qu'elle soit. Il semblerait que la nouvelle connexion entraine la suppression de l'attribut user de la session du premier utilisateur.

Quelqu'un sait d'où peut provenir le problème ?
Je ne sais pas si cela a un rapport, mais cette appli a été réinstallée plusieurs fois, et dernièrement sur un nouveau serveur, et il me semble que depuis le JSESSIONID n'apparait plus dans l'url. Est-ce normal ? Est-ce lié ?

Merci infiniment pour votre aide.
01/03/2011, 14h29
lunatix

a mon avis la disparition du jsession ID n'est pas en cause (ca doit passer en cookie). Tu peux valider en utilisant un navigateur qui refuse les cookies, le jsessionID doit revenir.

a vue de nez, tu dois bosser avec des actions qui ne sont pas threadSafe (struts1 ou spring MVC ou servlet par exemple) et quelque part referencer la session ou le user en attribut de servlet/action.
01/03/2011, 14h58
jgfa9

Merci pour ta réponse.
J'ai effectivement fait le test en déactivant les cookies et le JSESSIONID réapparait dans l'url... Mais plus rien ne marche car, si j'ai bien compris, il faudrait alors "encoder" les url (encodeURL()), ce qui n'est actuellement pas le cas...

Je travaille effectivement avec Struts 1.2.6.

Citation:

a vue de nez, tu dois bosser avec des actions qui ne sont pas threadSafe (struts1 ou spring MVC ou servlet par exemple) et quelque part referencer la session ou le user en attribut de servlet/action

Je ne comprends pas : tu as une suggestion pour corriger ce problème ?

Car par ailleurs, d'autres tests montrent que si 2 utilisateurs différents se connectent d'un même poste (j'ai fait le test), l'identifiant de session

Code:

request.getSession().getId()

reste le même (!?), ce qui est évidemment problématique si un seul utilisateur veut se connecter sous 2 profils différents au même moment (2 onglets par exemple). Je suppose qu'il doit bien y avoir un moyen de parvenir à cela...

Alors, comment faire ??

Merci !
01/03/2011, 15h18
tchize_

de mémorie (c'est loin) et vu que ca a l'air d'être du struts. Struts ne crée pas une instance de Action par requete, il en crée une seule qu'il réutilise. Donc toutes tes requetes partagent le même action, et donc le même champ User. C'est uniquement avec le Form que tu as la garantie d'une instance / requete ;)
01/03/2011, 17h02
OButterlin

Citation:

Envoyé par jgfa9

...
Car par ailleurs, d'autres tests montrent que si 2 utilisateurs différents se connectent d'un même poste (j'ai fait le test), l'identifiant de session

Code:

request.getSession().getId()

reste le même (!?), ce qui est évidemment problématique si un seul utilisateur veut se connecter sous 2 profils différents au même moment (2 onglets par exemple). Je suppose qu'il doit bien y avoir un moyen de parvenir à cela...

A vrai dire, c'est dans le même navigateur que tu as le partage de session.
Tu peux avoir 2 sessions avec 2 navigateurs (Firefox et IE par exemple) sur le même poste... mais bref, ton problème reste entier.

Ce que j'ai fait (il y a longtemps) est d'ajouter un identificateur de fenêtre qui est "trainé" partout (form + lien) pour tout accès à un contrôleur.
Plutôt que d'enregistrer dans la session, j'enregistre dans une Map<String, Map> dont la clé est cet identificateur de fenêtre. La Map est sauvegardée en session.
(Si ça peut t'aider...)

En clair en Struts1 ou spring mvc, les champs de type attributs sont partagés

exemple d'erreur classique en Struts1/springmvc/servlet en pseudo java
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 class action { private User user; <-- Ce champ est partagé par tout le monde.. d ou mélange public ... execute(request, response, form, etc..) { user = service.getUserFromLogin(...); session.setAttribute(User); } }

Merci beaucoup pour vos réponses !
Citation:
exemple d'erreur classique en Struts1/springmvc/servlet en pseudo java
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 class action { private User user; <-- Ce champ est partagé par tout le monde.. d ou mélange public ... execute(request, response, form, etc..) { user = service.getUserFromLogin(...); session.setAttribute(User); } }
Effectivement, lunatix, c'est exactement ce (l'erreur) que je fais...
Mais du coup, quelle est l'alternative ? Y-a-t'il une autre façon de faire ? Celle de OButterlin ?

Encore merci.

Tu peux déjà déplacer ta définition de User dans la méthode.

Code:

1
2
3
4
5
6
7
8
9
class action ...
{
    public ... execute(request, response, form, etc..) 
    {
        User user = service.getUserFromLogin(...);
        session.setAttribute(User);
    }
}

Citation:
Tu peux déjà déplacer ta définition de User dans la méthode.
Code:

1 2 3 4 5 6 7 8 class action ... { public ... execute(request, response, form, etc..) { User user = service.getUserFromLogin(...); session.setAttribute(User); } }
Oui mais je souhaitais pouvoir accéder à la variable user dans toutes les classes action précisément, ce qui n'est plus possible ici. Peut-être qu'il y a malentendu. Dans mon appli :
1. l'utilisateur se logue --> la variable user est mise en session
2. les classes actions héritent toutes d'une classe BaseAction qui est censée justement partager une variable user. Alors est-ce que c'est cette façon de faire qui est contestable (mettre une variable à disposition de toutes les classes) ?
Car il me semblait mieux d'instancier cette variable une fois pour toute que d'aller la récupérer à chaque fois dans la session...

Enfin, je ne comprends toujours pas pourquoi mais il semblerait que le problème de conflit de session provient d'une barre de progression (en Ajax) mise en place dernièrement : en effet, les traces indiquent clairement que lorsqu'un utilisateur utilise un écran qui affiche cette barre de progression (donc pour un calcul assez long), il y a ensuite confusion sur la variable username : voici un exemple de trace, ce sera peut-être plus clair :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 ... 03 mars 2011 12:35:52 (RightCalculationAction.java:143) - JG - calculate from 1/1/2010 to 31/12/2010 03 mars 2011 12:35:53 (RightCalculationAction.java:351) - contracts.size : 15925 03 mars 2011 12:35:53 (RightCalculationAction.java:371) - start calculation 03 mars 2011 12:35:53 (RightCalculationAction.java:431) - 0.0 contrats traités 03 mars 2011 12:36:18 (RightCalculationAction.java:431) - 1000.0 contrats traités 03 mars 2011 12:36:40 (RightCalculationAction.java:431) - 2000.0 contrats traités 03 mars 2011 12:37:04 (RightCalculationAction.java:431) - 3000.0 contrats traités 03 mars 2011 12:37:15 (ContractAction.java:1046) - JG - search contractid 17877 03 mars 2011 12:37:29 (RightCalculationAction.java:431) - 4000.0 contrats traités 03 mars 2011 12:37:54 (RightCalculationAction.java:431) - 5000.0 contrats traités 03 mars 2011 12:38:20 (RightCalculationAction.java:431) - 6000.0 contrats traités 03 mars 2011 12:38:42 (ContractAction.java:1046) - JG - search contractid 12181 03 mars 2011 12:38:47 (SearchArticleAction.java:533) - JG - sql (articles) : AND title LIKE('%éléments de classification%') 03 mars 2011 12:38:51 (RightCalculationAction.java:431) - 7000.0 contrats traités ...
L'utilisateur qui a lancé le calcul est JG. Mais ce n'est pas lui qui est à l'origine des traces en rouge !! C'est un autre utilisateur, mais c'est le username de JG qui est affiché.

Voici comment sont affichées les traces, dans ContractAction par exemple :

Code:

logger.info(username+" - search contractid "+contractForm.getContractid());

où username est une variable globale instanciée dans baseAction, la classe dont toutes les autres classes héritent :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 public abstract class BaseAction extends Action { protected static com.monappli.domain.User user = null; protected static String username = ""; ... public ActionForward execute(ActionMapping mapping, ActionForm form, HttpServletRequest request, HttpServletResponse response) throws IOException, ServletException { try { user = (com.monappli.domain.User) request.getSession().getAttribute("USER"); if (user != null){ username = user.getFirstname()+" "+user.getLastname(); } } } }
Et on pense que le problème vient de la barre car quand on la supprime, on n'a plus ce problème !
Le principe de cette barre est de faire des appels à intervalles réguliers vers le serveur pour voir où en est le calcul et mettre ainsi à jour l'affichage (et donc la progression de la barre). Ceci implique donc de multiples appels à la fonction prog depuis la page jsp :
Code:

1 2 3 4 5 function prog(){ xhr.open("GET", 'rightcalculation.do?actionType=PROGRESSPERCENT&counter='+Math.random()); xhr.onreadystatechange = function() { setProgressPercent(xhr); }; xhr.send(null); }
appelée ainsi :

Code:

itv = setInterval(prog, 100);

au moment où un utilisateur lance un calcul.

Ca fait beaucoup d'informations, mais est-ce que quelqu'un aurait déjà rencontré ce genre de problème ou saurait d'où il peut provenir ?

Merci infiniment pour votre lecture et vos réponses.

03/03/2011, 16h44
OButterlin
Mais comme ça a déjà été dit, ce n'est pas dans la classe Action qu'il faut partager mais dans la classe ActionForm.
Sinon, tu peux faire un getter dans ta classe générique qui ira chercher en session l'utilisateur... mais c'est moins bon... ça masque juste le
Code:

1 2 request.getSession().getAttribute("username");
03/03/2011, 16h58
jgfa9

Merci pour ta réponse rapide et excuse mon ignorance mais je ne comprends pas pourquoi on partagerait cette variable dans la classe qui hérite d'ActionForm alors que c'est dans les classes Action que j'en ai besoin...
Tu pourrais être un peu plus explicite ou me donner un exemple stp ?
Merci encore.
03/03/2011, 17h37
tchize_

parce que la classe Action sera utilisé en parallèle par plusieurs Thread, donc la valeur ne sera jamais la bonne que pour une seul Thread, tous les autres liront une mauvaise valeur. En plus tu utilise un champ statique, ce qui est encore pire, toutes les instance de Action paratageront la même valeur 8O