JAVA Runnable lambda에서 Servlets Thread Safety 이슈

개발 중 코드 중에서 Runnable lambda 문법을 이용해서 Thread 코드를 구현을 하였다. 

그런데 Thread 코드 내부에서 서블릿의 remote ip 코드가 일부 위치로 옮기면 ip정보가 유실하는 현상이 있었다. 

좀더 인터넷을 찾아보니, Servlet은 클래스 안에 멤버 변수를 사용하게 되면 모든 request에서 해당 변수를 공유 하기 때

문에 메서드 안에 지역 변수로 사용하라고 경고 되어 있다.

즉, Servlets은 클래쓰에 전역 변수를 사용하게 되면 session당 해당 변수를 공유하기 때문에 사용하면 안되고, Servelt 변

수를 사용하고 싶으면 메서드 지역변수에 저장해서 쓰라는 말이다.

public class ExampleServlet extends HttpServlet {

    private Object thisIsNOTThreadSafe;

    protected void doGet(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
        Object thisIsThreadSafe;

        thisIsNOTThreadSafe = request.getParameter("foo"); // BAD!! Shared among all requests!
        thisIsThreadSafe = request.getParameter("foo"); // OK, this is thread safe.
    } 
}

출처 : https://stackoverflow.com/questions/3106452/how-do-servlets-work-instantiation-sessions-shared-variables-and-multithreadi/3106909#3106909

How do servlets work? Instantiation, sessions, shared variables and multithreading

그래서 다음과 같이 Servlet에서 받은 원격지 주소를 메서드 안에 지역 변수로 초기화해서 사용하였다.

String remoteIp = req.getRemoteAddr();//중요!!!
if (returnCode == -22) {

  Runnable runnable = () -> {
    try {
      TCPSocketUtil tcpSocketUtil = new TCPSocketUtil(installDBProperty.getSocketAddress(),installDBProperty.getSocketPort());
      jObj.addProperty("event", SocketEventNames.LOGIN_LOCK.getEventName());
      jObj.addProperty("remote_ip", remoteIp);
      jObj.addProperty("userid", loginAccountVO.getUserId());
      jObj.addProperty("auth", "manager");
      jObj.add("uids", uidsArr);
      tcpSocketUtil.sendMsg(gson.toJson(jObj));
    } catch (Exception e) {
      e.printStackTrace();
    }
  };

  Thread thread = new Thread(runnable);
  thread.start();

}