Discussion :

[K-Kaï]

J'ai pour le controle de l'acces aux pages de mon site creer un Filtre de Servlet, qui donc effectue quelques vérifications avant d'autiriser ou non la visite de tel ou tel page web.

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
25
26
27
28
29
30
31
 
public void doFilter(ServletRequest req, ServletResponse res,
		FilterChain chain) throws IOException, ServletException {
		System.out.println("Youhou");
		HttpSession session = null;
		try {
 
			/*On verifie que l'user est authentifié (presence de son objet en Session)*/
			session = haveSession((HttpServletRequest) req);
			/*On recupère l'URI demandé*/
			String URI = ((HttpServletRequest)req).getRequestURI();
 
			//On teste si la variable personne qui doit être ds la session est un User ou un Admin
			if(session.getAttribute("personne") instanceof User){
				isUserPage(URI);
			}
			else if (session.getAttribute("personne") instanceof Admin) {
				isAdminPage(URI);
			}	
			chain.doFilter(req,res);
		}
		catch (IllegalArgumentException e)
		{
			envoiErreur.sendLoginError((HttpServletRequest)req,(HttpServletResponse)res,EnvoiErreur.SESSION_ERROR);
		}
		catch (IdentificationException e)
		{
			envoiErreur.sendError((HttpServletRequest)req,(HttpServletResponse)res,EnvoiErreur.RESTRICT_ERROR,null);
		}
 
	}

Le problème est au niveau du renvoi d'erreur que j'essayais de faire comme ceci :

Code :

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

1
2
3
4
5
6
7
8
9
 
void sendError(HttpServletRequest req, HttpServletResponse res, String errorType, String optionnalText) throws IOException{
		HttpSession s = req.getSession(true);
 
		s.setAttribute("errorMessage",errorType);
		s.setAttribute("option",optionnalText);
 
		res.sendRedirect(res.encodeRedirectURL("pages/error.jsp"));
	}

Or il n'est pas possible de renvoyer une page directement à partir du filtre, il faut utiliser un ResponseWrapper que l'on passera dans le chain.doFilter()...

Or j'ai beau avoir lu le tutoriel de sun dessus, je ne comprend vraiment pas comment ecrire mon responseWrapper afin qu'au final cela renvoi ma page d'erreur avec le message d'erreur.

Quelqu'un pourrait t'il m'aider dans l'ecriture de cette classe ?

[fabszn]

Hello,

Est ce qu'en levant une exception identifié cela ne peut pas faire l'affaire?

[K-Kaï]

Je ne pense pas, car le filtre apparement se contente de filter et passe le relais à la suite de la chaine ensuite :

Code :

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

1
2
 
chain.doFilter(req,res);

Et c'est la qu'il faut remplacer par

Code :

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

1
2
 
chain.doFilter(req,wrapper);

si j'ai bien compris, le problème c'est que même avec le tutos sun :
http://java.sun.com/products/servlet/Filters.html

Je ne comprends pas grand chose qui me permettrait de renvoyer ma page d'erreur ...

[fabszn]

Hello,

Est ce que tu ne pourrais pas positionner un indicateur dans la request indiquant à la page suivante (ou dans les couche abstraite du controleur) qu'il y a eu un problème dans au niveau du filtre et de conditionner la page suivante...?

[K-Kaï]

Cela ne fonctionnera pas car il faudra que je teste en debut de doGet() et doPost() la valeur de cette variable de la sorte :

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
protected void doGet(HttpServletRequest req, HttpServletResponse res) throws ServletException, IOException {
		int action = Integer.parseInt(req.getParameter("action"));
		if(!req.getAttribute("accessError").equals("")){
			envoiErreur.sendError(req,res,(String)req.getAttribute("accessError"),null);
			req.removeAttribute("accesssError");
		}
		
		switch(action){
			case 1 : { //Consultation de la liste globale des fichiers
				List<Fichier> listFic = gestionUser.getListFichiers();
				req.setAttribute("liste",listFic);
				res.sendRedirect(res.encodeRedirectURL("./pages/restricted/restrictedAdmin/listeFichiers.jsp"));
				break;
			}
			case 2 : {//Deconnexion
				req.getSession(false).invalidate();
				res.sendRedirect("./pages/index.html");
				break;
			}
		}
	}

et a ce moment le code de la methode continuera sont execution quand même ...

[fabszn]

Hello,

Tu pourrais avoir une classe mère au dessus de ta servlet (MyAbstractServlet).
et tu pourrais mettre ton controle à ce niveau là.
Comme cela si il y a un problème tu ne fais pas appel à la méthode doGet ou doPost. tu fais ta redirection immédiatement..

Je ne sais pas si j'ai été tres clair dans mes explications...