0

A ver he puesto un loginFilter en mi proyecto web java, y quiero redirigir al usuario al main.jsp o a error.jps dependiendo de si esta logueado o no. Entonces os explico el recorrido y lo que ocurre.

Desde login.jsp, en el action del formulario de inicio de sesión pone login(es el servlet), pero no va directo al servlet sino que antes de llegar al servlet, pasa por el filtro(cosa que me gustaría que fuese al revés, me facilitaría mucho las cosas). Indagando, he descubierto que en el filtro, todo el codigo que pones despues del chain.doFilter(request, response), se ejecuta despues de ir al servlet. Es decir, del login.jsp llega al filtro en el filtro , cuando llega al chain.doFilter, para y se va al login.java, aqui hace lo que tiene que hacer y vuelve al filtro. En el filtro ahora lo que hago es comprobar si hay user en la sesión(osea si esta logueado). y si no lo esta tiene que redirigir a error.jsp, si lo esta, lo manda al main.jsp.

El caso es que cuando intento hacer eso, me da la siguiente excepción:

java.lang.IllegalStateException: No puedo reenviar después de que la respuesta se haya llevado a cabo.

LoginFilter.class

public class LoginFilter implements Filter {
protected FilterConfig filterConfig;

   java.util.List revokeList; 


   public void doFilter(ServletRequest request, ServletResponse response, 
   FilterChain chain) throws java.io.IOException, ServletException {
       RequestDispatcher dd;




      HttpServletRequest req = (HttpServletRequest)request;
      HttpServletResponse res = (HttpServletResponse)response;
      HttpSession session =((HttpServletRequest) request).getSession();
      chain.doFilter(request, response); 
      User user = (User) session.getAttribute("user");                
      String url = ((HttpServletRequest)request).getRequestURL().toString();
      String urNo = "mani.jsp";





      UserBO uBO = new UserBO();

     if(url.equals("http://localhost:8080/com.calendar/login.jsp")||url.equals("http://localhost:8080/com.calendar/index.html")|| url.equals("http://localhost:8080/com.calendar/registro.jsp")) {


     }else {
         if(user != null) {

              if(uBO.recoveryWeb(user, urNo)) {//este metodo comprueva si el 
                                               //user tiene acceso a esa web                                                                  
              }else {                   
                  url = "login.jsp";                      
              }           

         }else {                 
             url = "index.html";                  
          }          
         dd=request.getRequestDispatcher(url);
         dd.forward(request, response);          
     }





   }




}

lo he llamado LoginFilter, pero no es un filtro especifico para el login, tengo que cambiarle el nombre.

Quiero que me sirva para todas las paginas de mi web. Necesito que pase primero por el servlet y en el servlet comprueba si existe el usuario o no, y ademas en tal caso añade el usuario a la sesión. por lo tanto cuando llega al filter por primera vez la sesión es nula aun, por eso quería que primero se ejecutase el servlet y luego el filtro pero no he podido.

1 respuesta 1

1

El problema que te indica es debido a que estás haciendo dos redirecciones, una de ellas al hacer el doFilter y otra, al realizar el forward. En el momento de realizar una redirección, debes realizar un return para que no continue la ejecución del método.

Por otro lado el planteamiento de bloquear por ruta no es correcto, mejor define a qué servlets se aplica el filtro para que no lo haga por esas que defines.

Yo lo enfocaría con algo así:

    public void doFilter(ServletRequest request, ServletResponse response, FilterChain chain)
        throws java.io.IOException, ServletException {

    HttpServletResponse res = (HttpServletResponse) response;
    HttpSession session = ((HttpServletRequest) request).getSession();

    User user = (User) session.getAttribute("user");
    String url = ((HttpServletRequest) request).getRequestURL().toString();
    String urNo = "mani.jsp";

    if (user == null || !uBO.recoveryWeb(user, urNo)) {// este metodo comprueba si el user tiene acceso a esa web
        res.sendRedirect("/ruta/login");
        return;
    }
    chain.doFilter(request, response);
}

Observa el return justo después de hacer la redirección.

En la definición del filtro:

<filter>
    <filter-name>LoginFilter</filter-name>
    <filter-class>url.tu.paquete.LoginFilter</filter-class>
    <init-param>
        <param-name>loginPage</param-name>
        <param-value>/login.jsp</param-value>
    </init-param>
</filter>

<filter-mapping>
    <filter-name>LoginFilter</filter-name>
    <servlet-name>TuServletSecurizado</servlet-name>
</filter-mapping>

<servlet>
    <description>TuServletSecurizado</description>
    <display-name>TuServletSecurizado</display-name>
    <servlet-name>TuServletSecurizado</servlet-name>
    <servlet-class>url.tu.paquete.TuServletSecurizado</servlet-class>
</servlet>

El que no quieras securizar, por ejemplo login.jsp no le añadas el filtro.

11
  • pero es que si lo hago así, el user es null, es decir primero hay que ir al servlet y después al volver, ejecutarse el if.
    – sergipc88
    Commented el 22 may. 2018 a las 15:59
  • ademas ese if no me sirve, porque el metodo recoveryWeb, tengo que llamarlo si User no es null. osea, que si es null hago una cosa y si no lo es hago otra, por eso es que tienen que ir en ifs distintos. Y otra cosa, no puedo usar el break, me da error me dice que no puedo usarlo fuera de un Switch.
    – sergipc88
    Commented el 22 may. 2018 a las 16:17
  • es que el if tiene que ir despues del chain.doFilter, porque tiene que ejecutarse al volver del servlet. no entiendo que quieres decir cuando dices: el planteamiento de bloquear por ruta no es correcto, mejor define a qué servlets se aplica el filtro para que no lo haga por esas que defines. precisamente trato de bloquear el acceso a según que rutas si una persona no esta logueada o no tiene el rango adecuado.
    – sergipc88
    Commented el 22 may. 2018 a las 16:24
  • Si lo hago como dices, el usuario va a ser null todo el rato, porque primero hay que ir al servlet a comprobar si el usuario existe, de ahi que ponga todo el codigo despues del chainDoFilter
    – sergipc88
    Commented el 22 may. 2018 a las 16:33
  • 1
    vale ahora te he entendido, muchas gracias
    – sergipc88
    Commented el 25 may. 2018 a las 9:32

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.