search

alibaba / hessian2-codec

hessian2-codec it is a complete C++ implementation of hessian2 spec
Apache License 2.0
26 stars 9 forks source link

The hessian2 specification has some errors and also some details not explained #25

Closed felix021 closed 1 year ago

felix021 commented 1 year ago

Not sure whether I should submit the issue here, but the specification is really confusing.

For example:

  1. binary: in the 3. Hessian Grammar section, 0x41 ('A') represents non-final trunk, while in 4.1 binary, it says x62 ('b') represents any non-final chunk

  2. string: in the 3. Hessian Grammar section, [x30-x34] <utf8-data> represents string of length 0~1023, while in 4.12. string, it's [x30-x33] b0 <utf8-data>, and has no further explanation.

I may add more cases later.

felix021 commented 1 year ago
  1. int: in the 3. Hessian Grammar section, single octet integers have range -0x10 ~ 0x3f, but in 4.5.1. Compact: single octet integers, it's -16 ~ 47 (0x2f)
felix021 commented 1 year ago

Ignore me. Seems not the right place for reporting problems of the spec.