H5与kbe大数据传送的bug及建议

小笙

bug:
版本9.几，websocket_protocal.cpp:312
原来的：
        msg_payload_length = ((uint64)(pPacket->data() + pPacket->rpos() + 0) << 56) |
                         ((uint64)(pPacket->data() + pPacket->rpos() + 1) << 48) |
                         ((uint64)(pPacket->data() + pPacket->rpos() + 2) << 40) |
                         ((uint64)(pPacket->data() + pPacket->rpos() + 3) << 32) |
                         ((uint64)(pPacket->data() + pPacket->rpos() + 4) << 24) |
                         ((uint64)(pPacket->data() + pPacket->rpos() + 5) << 16) |
                         ((uint64)(pPacket->data() + pPacket->rpos() + 6) << 8) |
                         ((uint64)(pPacket->data() + pPacket->rpos() + 7));

修复后：
        msg_payload_length = ((uint64)(*(pPacket->data() + pPacket->rpos() + 0)) << 56) |
                         ((uint64)(*(pPacket->data() + pPacket->rpos() + 1)) << 48) |
                         ((uint64)(*(pPacket->data() + pPacket->rpos() + 2)) << 40) |
                         ((uint64)(*(pPacket->data() + pPacket->rpos() + 3)) << 32) |
                         ((uint64)(*(pPacket->data() + pPacket->rpos() + 4)) << 24) |
                         ((uint64)(*(pPacket->data() + pPacket->rpos() + 5)) << 16) |
                         ((uint64)(*(pPacket->data() + pPacket->rpos() + 6)) << 8) |
                         ((uint64)(*(pPacket->data() + pPacket->rpos() + 7)));

取的地址移位，所以如果有大数据进来的话，那么会导致msg_payload_length 错误


建议：
1. H5的kbengine.js对大数据发送这块做得不够完善，发送大数据时，包体长度依旧是采用2个字节存
2. kbengine服务端对大数据接收不够友好，例如部分数据长度、数据大小的宏定义 定义得不够清晰，感觉是以前写代码中需要另外完善一些模块时，直接拿过来用，导致如果想要接收大数据时，那么是需要改宏定义的值，但是一旦改了，又牵动别的模块。

小笙

大佬是不是要送点论坛积分给我表彰下...  ^.^

柯标

已经提交了。

1：似乎， 目前的政策是不允许客户端提交超过65535字节的数据， 防止恶意传输。
这个只能后续考虑加个选项让服务器开放限制了
2：最早只能接收2字节数据长度，后来服务器内部补充了更大的长度机制。  这个只能你先看着改了， 后续再考虑放开支持

柯标

小笙 发表于 2018-9-13 11:12
大佬是不是要送点论坛积分给我表彰下...  ^.^

你的贡献都记住了， 很早的时候你找工作有人求推荐都推荐你了。

有贡献以后自然会有好处。
没有送积分选项。。