Spring中@Autowire注入HttpServletRequest的线程安全性问题咨询

首先可以明确告诉你：你现在的代码写法是线程安全的，不用担心多个请求之间的属性或请求头串掉的问题。下面给你拆解原因和注意事项：

为什么注入的HttpServletRequest是线程安全的？ #

Spring里通过@Autowired注入的HttpServletRequest并不是一个固定的实例，而是一个动态代理对象。背后的逻辑是：

• Spring会把HttpServletRequest标记为request-scoped（请求作用域）的bean，但因为Service默认是单例的，不能直接注入请求作用域的对象，所以Spring用了代理来解决这个矛盾。
• 每次你调用这个代理对象的方法（比如getHeader()、setAttribute()）时，它会从RequestContextHolder里获取当前线程对应的真实HttpServletRequest实例——而RequestContextHolder底层是用ThreadLocal实现的，每个线程的Request都是隔离的。

简单说就是：每个请求的线程拿到的都是自己专属的Request实例，操作的都是当前请求的属性和头信息，完全不会影响其他请求。

针对你的代码场景的验证 #

看你在IAMService里做的操作：

Long primaryKey = userDao.save(userdto,httpServletRequest.getHeader("loggedInUserId"));
httpServletRequest.setAttribute("userPrimaryKey",primaryKey);

这两步都是针对当前请求的：读取当前请求的loggedInUserId头，然后给当前请求设置userPrimaryKey属性。因为每个请求的线程都有自己的Request实例，所以不会出现线程A的属性被线程B覆盖的情况。

需要注意的坑（避免线程安全问题） #

虽然当前写法没问题，但有个绝对不能踩的坑：

• 不要把注入的HttpServletRequest保存到Service的成员变量中！比如：

  @Service
  public class IAMService {
      @Autowired
      private HttpServletRequest httpServletRequest;
      // 错误示例：把Request存到成员变量
      private HttpServletRequest savedReq;
  
      public String addUser(UserDto userdto){
          savedReq = httpServletRequest; // 这里会导致线程安全问题！
          // ...
      }
  }
  

  因为Service是单例的，savedReq会被所有线程共享，后续请求的线程会覆盖这个变量的值，导致线程之间的Request串用，这才会引发线程安全问题。只要你是直接在方法里使用注入的Request对象，就不会有这个问题。

备选方案（更直观的写法） #

如果你还是对代理对象有点不放心，可以换一种更直观的写法：在Controller方法里把HttpServletRequest作为参数传入，再传递给Service。比如：

@RestController
public class UserController {
    @Autowired
    IAMService iamservice;

    @PostMapping("/addUser")
    public String addUser(@RequestBody UserDto userdto, HttpServletRequest request){
        return iamservice.addUser(userdto, request);
    }
}

@Service
public class IAMService {
    @Autowired
    UserDao userDao;

    public String addUser(UserDto userdto, HttpServletRequest request){
        Long primaryKey = userDao.save(userdto, request.getHeader("loggedInUserId"));
        request.setAttribute("userPrimaryKey",primaryKey);
        return "User is added successfully";
    }
}

这种写法不需要在Service里注入Request，直接用方法参数，线程安全一目了然，也避免了对代理对象的理解成本。

内容的提问来源于stack exchange，提问作者vishal sundararajan