Статическая переменная ThreadLocal в WebApp - есть ли проблемы с безопасностью / производительностью? - PullRequest
Новая
       16

Статическая переменная ThreadLocal в WebApp - есть ли проблемы с безопасностью / производительностью?

12 голосов
/ 02 июня 2011

Я исследую и экспериментирую с переменной ThreadLocal в моем веб-приложении Java.Я использую переменную ThreadLocal для хранения имени пользователя (полученного из сеанса) перед запросом, а затем удаляю его после запроса.Я сделал это, вызвав статический служебный метод в ServletFilter.Причина, по которой я не просто извлекаю имя пользователя из сеанса, заключается в том, что я унаследовал систему с долго выполняющимися процессами, которые иногда выполнялись дольше, чем позволяло время ожидания сеанса.Моя идея состоит в том, чтобы получить имя пользователя до обработки запроса и сохранить его в переменной ThreadLocal, что дает мне доступ к имени пользователя на протяжении всего запроса, даже если это занимает более 15 минут.

Мой вопросявляется:

Есть ли какие-либо проблемы безопасности / производительности с этим дизайном, и если так, что было бы лучшим решением?Даже если нет проблем с безопасностью и / или производительностью, лучшие идеи приветствуются.Фрагменты из моего решения показаны ниже:

Вот мой служебный класс, который будет вызываться в моем фильтре, и везде, где мне потребуется имя пользователя. 

public abstract class UserUtil {
private static final ThreadLocal<String> threadUser = new ThreadLocal<String>();

public static String getUserId(){
    return threadUser.get();
}

public static void setUserId(String userId){
    threadUser.set(userId);
}

public static void removeUserId(){
    threadUser.remove();
}
}

Здесь используется мой фильтр сервлетустановить имя пользователя перед запросом (и очистить его через блок finally после запроса). 

public class UserFilter implements Filter {
public void init(FilterConfig filterConfig) throws ServletException {
}

public void destroy() {
}

public void doFilter(ServletRequest servletRequest, ServletResponse servletResponse, FilterChain filterChain) throws IOException, ServletException {
    try {
        HttpServletRequest request = (HttpServletRequest) servletRequest;
        UserBean userBean = (UserBean) ((HttpServletRequest) servletRequest).getSession().getAttribute("userBean");
        UserUtil.setUserId(userBean.getUserId());

        filterChain.doFilter(servletRequest, servletResponse);
    } finally{
        UserUtil.removeUserId();
    }
}

}

Вот моя конфигурация web.xml: 

<!--web.xml-->
<web-app>

...
...
...

<filter>
    <filter-name>UserFilter</filter-name>
    <filter-class>filter.UserFilter</filter-class>
</filter>
<filter-mapping>
    <filter-name>UserFilter</filter-name>
    <url-pattern>*.jsf</url-pattern>
</filter-mapping>

Любые идеи высоко ценится:)

1 Ответ

7 голосов
/ 02 июня 2011

На самом деле это довольно распространенный подход для присоединения информации о безопасности к потоку выполнения (или любой другой информации, связанной с выполнением).

Он используется внутри серверов Java EE, а также сторонними кодами / утилитами, такими как Spring.

Он будет работать просто отлично.

...