为什么我要避免在servlet中使用InheritableThreadLocal？

Question

我InheritableThreadLocal在Servlet班上使用.因此它可以从它的子线程中获得.InheritableThreadLocal在线程池执行器中使用这是邪恶的吗？.比如servlet线程池.

我的问题.

1)为什么要避免InheritableThreadLocals在servlet中使用？

2)这种内存泄漏是否可能InheritableThreadLocal？

3)有没有替代方案InheritableThreadLocal？

4)如果线程被重用会发生什么,存储的值threadlocal不会被清除？

我的实时场景

public class UserAccessFilter implements javax.servlet.Filter {

      static final InheritableThreadLocal<String> currentRequestURI = new InheritableThreadLocal<String>();

      public void  doFilter(ServletRequest req, ServletResponse resp , FilterChain fc) throws IOException, ServletException{
              String uri = request.getRequestURI();
              fc.doFilter(request, response);         
      }
}


public class MailServlet extends HttpServlet{

      @Override
protected void doGet(HttpServletRequest req, HttpServletResponse resp) throws ServletException, IOException {
    String mailCount = req.getParameter("mailCount");

    if(mailCount != null && !"".equals(mailCount) && mailCount.matches("[0-9]+")){
        MailThread mailThread = new MailThread("xxx@gmail.com", generateToAddress(Integer.parseInt(mailCount))); //NO I18N
        Thread t = new Thread(mailThread);
        t.start();
    }

    resp.getWriter().println("Mail Servlet.............."); //NO I18N

}              

}

class MailThread implements Runnable{

private String from;
private String to;

public MailThread(String from , String to){
    this.from = from;
    this.to = to;
}


@Override
public void run() {
    sendMail();
}

    public void sendMail(){
        //I want this uri from child threads. I can't pass this value to this constructor.
        String uri = currentRequestURI.get();
        //Do Mail Operations
    }


}

过滤器 - > Servlet A - >子线程--->邮件线程(这里我得到过滤器中设置的值).

Answer 1

为什么我们要避免在servlet中使用InheritableThreadLocals？

它们代表了将信息从一个请求泄漏到另一个请求的潜在途径."问题"是请求由线程池处理.当请求完成时,线程处理的下一个请求可能是针对不同的用户.但是如果你在完成第一个请求时忘记清除线程本地状态,那么第二个请求可能会使用它.

InheritableThreadLocal中是否存在内存泄漏？

是的......有点儿.如果我们假设工作池是有界的,则任何线程的线程本地状态都可能被覆盖,从而清除内存泄漏.在最坏的情况下,问题是有限的内存泄漏......受池中线程数量的限制.

信息泄漏问题更令人担忧.

是否有任何替代InheritableThreadLocal？

在请求或响应对象中设置属性更好.

如果线程将被重用会发生什么,threadlocal中存储的值将不会被清除.

它不会被清除.那就是问题所在!

Answer 2

您的示例有效，MailThread 继承了currentRequestURI创建时的值。

但是 theUserAccessFilter和 thejava.lang.InheritableThreadLocal只会混淆代码想要做什么，这是邪恶的部分。

更改构造函数MailThread以便您可以像这样传递请求 URI有什么问题MailServlet：

MailThread mailThread = new MailThread("xxx@gmail.com", generateToAddress(Integer.parseInt(mailCount)), req.getRequestURI());

那么你就不需要本地线程，不需要过滤器，代码也更清晰。