序列化引发的血案

问题背景

早上来上班,突然值班群里有人报问题,说店铺属性在JD App前端显示错乱了。打开App,点开一个店铺查看,问题复现。打开内部店铺信息查询系统,发现查出来的属性确实有一部分错乱了。OMG~线上bug来了。
查看JONE系统,发现昨晚有人加班上线了新功能,甭问,肯定是这次上线造成的问题。尝试回滚一台线上机器,问题解决。来吧,全量回滚吧!悲催的绩效啊~

问题排查

1.店铺信息查询逻辑

App中的店铺信息是调用微服务shop-base的JSF(类似Dubbo)接口查询店铺基础信息的。这次问题也就是出在这个shop-base上。这个接口的入参中,传入的是一个需要查询的属性List,这些属性被定义成VenderAttributeEnum,也就是List< VenderAttributeEnum >。调用方把需要查询的属性指定好,塞入list传进接口。
下面是接口定义(省去无关的参数):

    public Result getShopAttributesByShopId(long shopId, List<VenderAttributeEnum> QueryShopInfoResult, etc...);

2.问题原因

询问了昨晚的开发者,新功能迭代中,增加了两个新属性。打开git日志,发现VenderAttributeEnum有修改,立刻查看修改记录,发现新增了两个属性。问题就在这里。VenderAttributeEnum的定义类似如下,新增了两个属性:

public enum VenderAttributeEnum {
    /*店铺Logo*/
    VENDER_SHOP_LOGOXXX, 
    /*店铺Id*/
    VENDER_SHOP_ID, 
    /*店名*/
    VENDER_SHOP_NAME, 
    /*店铺状态*/
    VENDER_SHOP_STATUS,
    /*新增属性1*/
    VENDER_NEW_ATT1,    -----------> 新增1
    /*新增属性2*/
    VENDER_NEW_ATT2,    -----------> 新增2
    /*Logo图片URI地址*/
    VENDER_SHOP_LOGO_XXX,      -----------> 开始错乱
    ......(省略几万个属性)

看上去这样修改没啥问题,但是这两个属性被加在了中间,而不是所有属性的末尾。经过确认,发现出现错乱的是从属性VENDER_SHOP_LOGO_XXX开始的,进一步验证了此处修改造成了错乱。

为什么加在了Enum中间就会出现错乱呢?

刚进公司的时候,自己学习过JSF的文档。其中提到过对于Bean和Enum的定义,属性顺序不能变化。因为JSF默认使用Msgpack做序列化。其特点就是序列化后字节数变小,而变小的原因数据压缩,其追求的是更小、更快。为了更小,MsgPack序列化的结果中只包含了value,而不包含key。因此,在进行反序列化的时候需要保证类中属性的顺序必须保证跟序列化时完全一致

3.解决方法

  • Msgpack是个坑,考虑使用hessian、protobuf,不再对属性的顺序有要求。也许后面还有人犯同样的错误。到底使用哪个,性能比较后决定。
  • 接口测试的时候要关注所有属性的值,而不是只测试新增的属性。自动化回归测试是个好方法。
  • 暴露新员工培训不足,对使用到的中台服务逐一培训学习。

教训是深刻的,千万不要再犯!

©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 215,634评论 6 497
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 91,951评论 3 391
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 161,427评论 0 351
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 57,770评论 1 290
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 66,835评论 6 388
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 50,799评论 1 294
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 39,768评论 3 416
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 38,544评论 0 271
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 44,979评论 1 308
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 37,271评论 2 331
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 39,427评论 1 345
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 35,121评论 5 340
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 40,756评论 3 324
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 31,375评论 0 21
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 32,579评论 1 268
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 47,410评论 2 368
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 44,315评论 2 352

推荐阅读更多精彩内容

  • JAVA序列化机制的深入研究 对象序列化的最主要的用处就是在传递,和保存对象(object)的时候,保证对象的完整...
    时待吾阅读 10,859评论 0 24
  • 前言 首先本文不讨论为什么要服务化,包括服务化的优点缺点。其次本文也不讨论什么是微服务,也不讨论微服务和SOA的区...
    章耿阅读 2,472评论 1 16
  • 国家电网公司企业标准(Q/GDW)- 面向对象的用电信息数据交换协议 - 报批稿:20170802 前言: 排版 ...
    庭说阅读 10,949评论 6 13
  • 在Java中,我们可以通过多种方式来创建对象,并且只要对象没有被回收我们都可以复用该对象。但是,我们创建出来的这些...
    懒癌正患者阅读 1,528评论 0 12
  • 时间进入初夏,坐出租车去火车站接女儿的大凡,此刻双目呆滞,内心里满是大赚之后一分不剩的极度空虚,压抑得像此刻的天空...
    蓝小蓟阅读 501评论 1 0