NodeBB聊天消息通知内存溢出问题分析与修复

问题背景

NodeBB论坛系统中存在一个与聊天消息通知相关的严重问题：当公共聊天室在短时间内发送大量消息且用户启用了邮件通知功能时，系统会出现消息内容重复叠加的现象。这会导致生成的电子邮件内容异常庞大，最终引发Node.js进程内存耗尽而崩溃。

问题现象

系统日志显示以下关键错误信息：

1. RangeError: Invalid string length - 字符串长度超出限制
2. FATAL ERROR: Reached heap limit Allocation failed - JavaScript heap out of memory - 堆内存耗尽
3. 崩溃发生在notifications.js文件的pushToUids函数中

技术分析

问题根源

经过深入分析，发现该问题由两个关键因素共同导致：

1. 消息累积机制缺陷：当新消息到达速度快于系统配置的notificationSendDelay（默认60秒）时，系统会不断累积消息内容而不发送。这种设计导致bodyLong字符串长度无限增长。

2. 引用传递问题：当聊天室用户数超过500人时，系统会触发分批处理机制。由于通知对象是通过引用传递的，在pushToUids函数中被修改后，每次调用都会导致消息内容重复拼接，形成指数级增长。

关键代码分析

问题主要出现在通知系统的核心处理逻辑中：

• 消息内容拼接时没有长度限制检查
• 通知对象作为引用传递导致重复修改
• 大批量用户处理时没有考虑内存占用问题

解决方案

修复方案包含以下关键改进：

1. 引用传递问题修复：确保每次调用pushToUids时处理的是通知对象的副本而非引用，避免重复修改。

2. 消息发送机制优化：调整通知发送逻辑，确保即使新消息持续快速到达，系统也能按时发送累积的通知，避免无限增长。

3. 内存安全增强：在处理大批量用户时增加内存使用监控和限制，防止内存耗尽。

实施效果

经过修复后：

• 消息通知内容不再出现重复叠加现象
• 系统内存使用保持稳定
• 大规模聊天室场景下的可靠性显著提升

最佳实践建议

对于运行NodeBB系统的管理员，建议：

1. 定期更新到最新版本，获取稳定性修复
2. 对于大型社区，适当调整notificationSendDelay参数
3. 监控系统内存使用情况，特别是聊天功能活跃时段
4. 考虑对超大规模聊天室启用消息频率限制

该修复不仅解决了特定场景下的崩溃问题，还提升了整个通知系统的健壮性，为高并发聊天场景提供了更好的支持。