Statamic表单提交重复执行问题的排查与解决

问题现象

在使用Statamic构建的表单系统中，开发人员遇到了一个奇怪的问题：表单控制器的submit方法被重复执行，导致邮件被重复发送。尽管前端只发起了一次AJAX请求，但后端日志显示FormController的submit方法被调用了两次。

排查过程

开发人员首先进行了以下排查步骤：

1. 前端验证：确认AJAX请求确实只发送了一次，排除了前端重复提交的可能性
2. 邮件队列检查：验证了自定义的MailJob实现，确认问题不是由邮件队列引起的
3. 环境测试：在全新安装的Statamic环境中无法复现该问题
4. 日志追踪：通过详细的日志记录发现请求确实被处理了多次

深入分析

通过更详细的日志记录，发现了以下调用链：

1. 初始AJAX请求正常到达控制器
2. 控制器调用SendEmails分发邮件任务
3. 随后又出现第二次"AJAX调用"的日志记录
4. 邮件任务被重复添加到队列中

值得注意的是，在浏览器开发者工具的网络面板中，确实只观察到一个请求记录，这排除了前端重复提交的可能性。

根本原因

经过深入排查，发现问题出在一个自定义中间件上。该中间件原本用于设置默认语言环境，但没有正确忽略Statamic的动作路由。更关键的是，中间件中存在以下问题代码：

if ($next($request)) {
    // 中间件逻辑
}

这种写法直接执行了$next($request)闭包作为条件判断，导致请求被提前处理一次，然后在中间件完成后又被正常处理一次，从而造成了重复执行的现象。

解决方案

修复该问题的方案包括：

1. 修改中间件逻辑，避免直接执行$next($request)作为条件
2. 确保中间件正确处理Statamic的特殊路由
3. 添加适当的路由过滤条件，避免中间件影响Statamic的核心功能

经验总结

这个案例提供了几个重要的经验教训：

1. 中间件设计要谨慎：中间件中的条件判断不应直接执行请求处理闭包
2. 日志记录很重要：详细的日志记录是诊断这类问题的关键
3. 环境差异排查：在新环境中无法复现问题时，应该重点检查环境特有的配置和自定义代码
4. 框架路由特性：使用全栈框架时，需要了解其对特殊路由的处理方式

最佳实践建议

为了避免类似问题，建议：

1. 在编写中间件时，使用Reflection等工具来检查路由信息，而不是直接执行闭包
2. 为Statamic的特殊路由添加明确的排除规则
3. 实现全面的日志记录，特别是在关键的处理流程中
4. 在修改核心功能前，先在隔离环境中测试变更

这个问题虽然最终发现是自定义代码导致的，但排查过程展示了系统化调试的重要性，也为其他Statamic开发者提供了有价值的参考经验。