起因:
当前系统需要一个短信通知功能,后台给了我短信发送接口。按照往常一样前端获取用户勾选的过滤条件,post提交给该接口。由于测试数据库中存的是用户的真实手机号,导致我前端在调通该功能时,对符合条件的近千个人员发送了数量不等的测试短信。
原因判断:
首先短信发送的MQ是老大搭建的,甲方有提供一个短信发送的接口之类的东西,在老大配好基本的之后,事实上后台同事只需要干一件很简单的事,就是获取我前端传入的过滤参数及短信内容,在数据库中查询符合条件的人员,放入一个对象集合中,这个短信发送对象包含手机号,人员姓名及短信文本。
最后一个sendMessage方法就完成发送了。
messageSender.sendMessage(XmlSeriesUtils.toXml(sd));
看到这,就知道原因了
DB 存储是人员真实手机号,只要该人员被查出来,那就一定会被发送出去,因为在这里后端人员并没有对发送人员的做特殊处理。
总结反思:
1.测试数据库不要存储真实人员数据(即系统上线后,所用到的真实数据)。
2.后台调用发送短信服务时,因对发送数据进行基本的判断,无论是代码的,还是人员思维上的。
3.前后端交流时,接口文档一定要提供并且描述清楚。
4.单元测试、测试、测试!重要的事情说三遍,事实上该接口并未经过严谨的单元测试,后端人员确实完成了发送的功能,但并未考虑到这是真实数据,可能他知道开发的时候,将发送人员做一个处理,但提供的正式接口,确实查出所有人员都可以发送。
目前业务处理直接放在了controller,并未放在service,所以也就没了单元测试,最终暴露出来的问题就是直接在前端展示了。
5.干活不要赶,不要急,因为是前后端分离一定要和后端人员交流清楚。