Discussion :

[enjoy]

Bonjour à tous, je développe une application Web s'appuyant les frameworks Hibernate, Acegi et Icefaces...

J'utilise bien sur un système de logging pour s'authentifier...
Hors j'ai besoin dans mon application de STOCKER l'object User qui s'est loggué.
C'est pourquoi j'ai crée une classe :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
public class ThreadLocalUser {
	    private static final ThreadLocal<User>  name = new ThreadLocal<User> ();
		private static final ThreadLocal<LdapUserDetails> userDetailz = new ThreadLocal<LdapUserDetails>();
 
 
	    private ThreadLocalUser() {
	    }
 
	    public static User getUser() {
	        return (User) name.get();
	    }
 
	    public static void setUser(User user) {
	        ThreadLocalUser.name.set(user);
	    }
 
	    public static LdapUserDetails getDetails() {
	        return (LdapUserDetails) userDetailz.get();
	    }
 
	    public static void setDetails(LdapUserDetails details) {
	        ThreadLocalUser.userDetailz.set(details);
	    }
	}

J'initialise le ThreadLocal au moment de se logguer sans problème...

Mais ensuite lorsque, l'application se charge ou lorsque je navigue dedans soudainement le ThreadLocal devient null et je n'ai plus accès au User...
Quelqu'un pourrait-il m'éclairer??

[Duc Lebowski]

Un ThreadLocal n'est propagé que le long d'un seul Thread principal. Lorsque le Thread principal termine son exécution, le ThreadLocal est détruit par la même occasion.

Chaque requête utilisateur (HttpServletRequest) représente un Thread différent, donc....

Il faut que tu renseigne ton User à chaque fois, par exemple dans un Filter.

[adiGuba]

Salut,


Dans une application Web il serait plus propre d'utiliser le scope session pour stocker ces informations...

a++

[enjoy]

Mon thread local est stocker dans un BEAN que je déclare dans un fichier xml Spring...
C'est ce Bean qui récupère les informations lors du logging...et donc le User...
Il faut donc que je le stocke dans un objet du faces-config.xml plutot??

[Duc Lebowski]

Celà ne changera rien à ton problème, à la fin de chaque requête HTTP ton ThreadLocal sera détruit.

Pourquoi ne propages-tu pas User tout simplement plutôt que de passer par un ThreadLocal ? Comme le dit adiGuba, généralement on stocke User dans la session.

Maintenant si tu veux vraiment passer par un ThreadLocal pour pouvoir appeler User à nimporte quel endroit, une solution que j'avais mise en place sur un projet était de passer par un filtre qui va "accrocher" User en tant que ThreadLocal avant d'exécuter chaque requête :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
public class UserFilter implements Filter
{
  public void doFilter(ServletRequest request,
                       ServletResponse response, 
                       FilterChain chain) 
    throws IOException, ServletException
  {
    HttpServletRequest httpreq = (HttpServletRequest)request;
    HttpSession session = httpreq.getSession();
 
    if (session.getAttribute("user") == null)
    	session.setAttribute("user", new User("toto"));
 
    User user = (User)session.getAttribute("user");
    ThreadLocalUser.setUser(user);
 
    chain.doFilter(request, response);
  }
 
  public void init(FilterConfig config) throws ServletException {}
  public void destroy() {}
}

[enjoy]

Merci pour l'aide

Mais je veux bien stocker le User dans la session si cela est plus simple et que cela marche...Mais je ne sais pas comment faire...
Dans ce cas, on parle bien de la Session au sens HTTP?? Car Session peut avoir bcp de sens différents...(hibernate etc)
Pourriez vous m'indiquez comment faire svp?

[Duc Lebowski]

Oui, il s'agit bien de la session HTTP.

Mais là je vois que tu es en JSF et je ne suis pas un expert sur cette techno.

Tu peux récupérer la session en faisant ça mais je ne pense pas que ce soit une bonne pratique en JSF :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
FacesContext context = FacesContext.getCurrentInstance();
HttpSession session = (HttpSession)context.getExternalContext().getSession();

Après ça dépend...

comment ton User est-il créé ?

comment est-il référencé dans faces-context ?

pour qu'elle raison dois-tu le stocker ?

[enjoy]

Je le stocke car j'ai des préférences utilisateurs sur chaque onglet, qui permettent de ne pas afficher toutes les colonnes + l'ordre...

Mais je viens de tester quelque chose et cela semble fonctionner...
C'est en rapport avec le context Acegi en fait puisque jutilise LdapAuthoritiesPopulator pour le logging

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
	public static User getUser() {
 
		return managementService
				.getUserFromUserLogin(((LdapUserDetails) (SecurityContextHolder
						.getContext().getAuthentication().getPrincipal()))
						.getUsername());
	}

Cela semble bien marché...
Merci pour votre aide, elle m'a bcp guidé

[Duc Lebowski]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
	public static User getUser() {
		...}

Ta méthode est statique, tu es bien sûr que tes utilisateurs ne vont pas partager le même User ?