Express启动报Cannot read properties of undefined(indexOf)修复方法

你遇到的TypeError: Cannot read properties of undefined (reading 'indexOf')报错，根源是req.get('X-Forwarded-Proto')的返回值为undefined。
X-Forwarded-Proto是反向代理（Nginx、CDN、云负载均衡等）才会添加的请求头，本地开发环境、或者反向代理没有正确配置转发该头时，调用req.get('X-Forwarded-Proto')就会拿到undefined，此时直接链式调用indexOf方法就会抛出类型错误。

方案1：兼容原有逻辑，增加空值兜底 #

在调用indexOf之前先判断请求头是否存在，不存在时用空字符串兜底，同时增加本地HTTPS环境的判断，避免无代理场景下的死循环重定向：

function checkHttps(req, res, next){
  // 协议检查，http请求重定向到https
  const forwardedProto = req.get('X-Forwarded-Proto') || ''
  if(forwardedProto.indexOf("https") !== -1){
    return next()
  } else {
    // 无代理场景下直接判断当前协议，避免死循环
    if (req.protocol === 'https') return next()
    res.redirect('https://' + req.hostname + req.url);
  }
}

方案2：使用Express原生能力（推荐） #

Express本身内置了反向代理场景下的协议识别能力，不需要手动解析X-Forwarded-Proto头，稳定性更高：

1. 首先在Express初始化逻辑中（路由注册之前）添加信任代理配置：

// app是你初始化的express实例，比如const app = express()
app.set('trust proxy', true)

2. 简化HTTPS检查中间件，直接使用Express内置的协议判断属性：

function checkHttps(req, res, next){
  // req.secure是Express内置属性，等价于req.protocol === 'https'，会自动识别代理转发的协议头
  if(req.secure){
    return next()
  }
  res.redirect('https://' + req.hostname + req.url);
}

这种写法完全避免了手动取请求头出现undefined的问题，也是Express官方推荐的HTTPS重定向实现方式。

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