Axios 1.7.x版本中progressEventReducer.js的类型错误问题解析

问题背景

在Axios 1.7.3和1.7.4版本中，部分开发者遇到了控制台报错"TypeError: t is not a function"或"TypeError: e is not a function"的问题。这些错误源自progressEventReducer.js文件，且只在这些新版本中出现，1.7.2及更早版本则表现正常。

错误原因分析

这个问题的根源在于Axios 1.7.3版本中对上传进度事件处理逻辑的修改。具体来说，在commit e3c76fc中，开发团队修改了onUploadProgress参数的检查方式：

旧版本代码：

if (typeof _config.onUploadProgress === 'function' && request.upload) {

新版本代码：

if (onUploadProgress && request.upload) {

这个看似简单的修改实际上改变了参数验证的严格程度。旧代码使用typeof操作符明确检查参数是否为函数类型，而新代码仅进行真值检查。这意味着任何非空、非零、非false的值都会通过检查，即使它不是函数。

实际影响

当开发者传递一个非函数但为真值的对象作为onUploadProgress参数时（例如空对象{}），新版本的检查会通过，但在progressEventReducer.js中尝试调用这个"函数"时就会抛出类型错误。

这种情况常见于以下场景：

1. 开发者设置了默认配置对象，其中包含空对象{}作为onUploadProgress的初始值
2. 只有在需要上传进度功能时才将其设置为实际函数
3. 在1.7.2及更早版本中，这种模式可以正常工作，因为typeof检查会过滤掉非函数值

解决方案

针对这个问题，开发者可以采取以下几种解决方案：

1. 修改默认配置：将onUploadProgress的默认值从空对象改为false或null

// 不推荐的旧方式
const defaultConfig = {
  onUploadProgress: {}
};

// 推荐的新方式
const defaultConfig = {
  onUploadProgress: false
};

2. 升级代码逻辑：仅在需要时添加onUploadProgress配置，而不是设置默认空对象

3. 回退版本：如果暂时无法修改代码，可以暂时回退到1.7.2版本

最佳实践建议

为了避免类似问题，建议开发者在处理回调函数参数时：

1. 始终使用typeof检查来验证函数类型
2. 避免使用真值检查作为函数验证的唯一手段
3. 对于可选回调参数，使用null或undefined作为默认值，而不是空对象

总结

这个问题的出现提醒我们，即使是看似简单的真值检查替换，也可能在特定场景下引入意外的行为变化。Axios作为广泛使用的HTTP客户端库，其API的细微变化都可能影响大量现有代码。开发者在升级依赖版本时应当仔细阅读变更日志，并对可能受影响的功能进行充分测试。