Redis初始化函数仅调用一次却疑似内存泄漏，求原因解析

我来帮你拆解这个内存泄漏的问题～

首先先看你提供的代码：

const redis = require('redis');

module.exports = function redisClientConfig(app) {
  const redisClient = redis.createClient(app.get('redis-config'));

  redisClient.on('connect', () => {
    app.redisClient = redisClient;
  });
};

你的疑问是：这个初始化函数只调用一次，却引发了内存泄漏，猜测是connect事件触发时app.redisClient和redisClient形成闭包导致的。

问题根源解析 #

其实核心问题不是闭包本身，而是connect事件的触发特性+永久事件监听器的组合：

• Redis客户端并非只触发一次connect事件——当网络波动、服务重启等情况导致连接断开后，客户端会自动重连，每次成功重连都会再次触发connect事件。
• 你用on绑定了一个永久的事件监听器，这个监听器内部引用了redisClient和app实例。由于app（比如Express的app对象）是长期存活的全局实例，它会一直持有这个监听器的引用，而监听器又持有redisClient的引用，导致redisClient实例永远无法被垃圾回收（GC）。
• 哪怕你每次重连都给app.redisClient赋值同一个redisClient，旧的监听器依然存在，相当于一直在占用内存，时间久了就会显现出泄漏问题。

修复方案 #

针对这个场景，有两种简单有效的修复方式：

方案1：只监听一次connect事件

如果你的需求只是在首次连接成功时给app挂载redisClient，直接用once代替on——这个方法会在事件触发一次后自动移除监听器，不会持续持有引用：

const redis = require('redis');

module.exports = function redisClientConfig(app) {
  const redisClient = redis.createClient(app.get('redis-config'));

  // 用once确保监听器仅执行一次，执行后自动销毁
  redisClient.once('connect', () => {
    app.redisClient = redisClient;
  });
};

方案2：手动移除监听器（如需处理重连）

如果需要每次重连都执行逻辑，那可以在监听器内部手动移除自己，或者确保不会重复绑定：

const redis = require('redis');

module.exports = function redisClientConfig(app) {
  const redisClient = redis.createClient(app.get('redis-config'));

  function handleConnect() {
    app.redisClient = redisClient;
    // 若无需后续监听，执行后立即移除监听器
    redisClient.off('connect', handleConnect);
  }

  redisClient.on('connect', handleConnect);
};

额外提醒：如果使用的是Redis客户端v4+版本，createClient的API有一些变化，但事件监听的核心逻辑是一致的，依然要注意避免永久监听器导致的内存泄漏问题。

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