WÁNG Xuěruì
WÁNG Xuěruì
Have you tested this already? I can't reproduce the problem so I can only trust your fix, but it looks good superficially. Do you need a release after I merge...
不清楚,我们这边没有这种场景,建议自己试一下
也没试过。。
Why? 考虑 `block_size != 32` 的通用情况么?
问下这个情况是近期出现的吧?以前记得没有。这种时候确实是希望返回那个“未指定”吗?我已经忘了“未指定”是不是一个合法的性别了(正常操作会不会出现)。
> @xen0n 是最近出现的,如果不改的话,会报错 Ack 看起来是疼讯的文档不讲究喔
没有测试过,这应该完全是疼讯的业务逻辑,我们公司只有不多的内部应用接入了,没有碰到限流之类问题
有文档吗?给的链接是个落地页。 由于设计之初就预见到了有变更 API host 的需求,绝大部分代码不用动。除非动到了核心公共流程(鉴权流程、请求响应格式、错误码),否则直接往现有接口加就好了。不需要做任何区分对待,在 SaaS 企业微信上调用私有部署 only 的接口就让他报错就好了。
相同的接口请求参数名字不一样?优先跟企业微信的人反馈 bug 吧。。这不像是符合一般业务研发常识的 API 设计。需要给出必须这么做(给下游对接人为增加难度)的原因。他们能改掉是最好。否则可能需要起两个内部接口(`execXxx` 的 API 写法,以及相应 model 类型)然后想办法区分。这样的话一个 flag 字段也不可避免了。难受。
Feature request 需求合理,主要矛盾是需要找时间做掉= =