忽略ViewExpiredException是错误的吗?

时间:2015-02-23 13:50:46

标签: jsf java-ee viewexpiredexception

一周前,我研究过ViewExpiredException,并且我已经阅读了几个相关内容。

我的问题,在某些情况下,我想忽略ViewExpiredException。这些情况不需要“会话”,我的Beans是@RequestScoped。例如,网页login.xhtmlregister.xhtmlpasswordRecovery.xhtml

在这些情况下,向用户说明您的会话已过期是非常奇怪的。因此,如果您打开登录页面并静止一段时间,当他通知您的数据并单击“登录”时,它将被转发到错误页面。我会忽略它并让用户透明。

所以,到目前为止,我的解决方案是创建一个ExceptionHandler来忽略这些例外:

@Override
public void handle() throws FacesException {
    for (Iterator<ExceptionQueuedEvent> i = getUnhandledExceptionQueuedEvents().iterator(); i.hasNext();) {
        ExceptionQueuedEvent event = i.next();
        ExceptionQueuedEventContext context = (ExceptionQueuedEventContext) event.getSource();
        Throwable t = context.getException();
        // just remove the exception from queue
        if (t instanceof ViewExpiredException) {
            i.remove();
        }
    }
    getWrapped().handle();
}

然后,我创建了一个过滤器来检查用户是否已登录,如果没有,则重定向到登录页面(此过滤器仅应用需要身份验证的页面):

@Override
public void doFilter(ServletRequest req, ServletResponse res, FilterChain chain)
        throws IOException, ServletException {
    HttpServletRequest request = (HttpServletRequest) req;
    HttpServletResponse response = (HttpServletResponse) res;

    if (!loginManagedBean.isLogged()) {
        String pathLogin = request.getContextPath() + "/" + LOGIN_VIEW;
        if (isAJAXRequest(request)) {
            response.setContentType("text/xml");
            response.getWriter()
                    .append("<?xml version=\"1.0\" encoding=\"UTF-8\"?>")
                    .printf("<partial-response><redirect url=\"%s\"></redirect></partial-response>", pathLogin);
            return;
        }

        pathLogin += "?source=" + request.getServletPath();

        response.sendRedirect(pathLogin);
        return;
    }

    chain.doFilter(request, response);
}

因此,当会话到期时,不会影响登录和注册页面中的用户体验。在我希望会话的页面上,由过滤器处理。

那会是一个很好的解决方案吗?忽略ViewExpiredException中的ExceptionHandler是否存在任何安全风险?

1 个答案:

答案 0 :(得分:2)

在这种特定情况下,忽略它们在技术上并不坏,但它表明设计不好。好像你正在使用错误的工具来完成这项工作。即这些观点实际上永远不会过期。

只要具体说明那些无国籍的观点。

<f:view transient="true">
    ...
</f:view>

这可以放在页面的任何位置,甚至是重复的,但大多数自我记录使它成为页面,组合或定义的顶级标签。

另见: