Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

another version hessian2 support #32

Open
wbpcode opened this issue Apr 17, 2024 · 4 comments
Open

another version hessian2 support #32

wbpcode opened this issue Apr 17, 2024 · 4 comments

Comments

@wbpcode
Copy link
Collaborator

wbpcode commented Apr 17, 2024

A helpless fact is that there are two different versions of hessian2. One is the version that the dubbo used. See http://hessian.caucho.com/doc/hessian-serialization.html.

Another is the version the sofa used. See https://www.caucho.com/resin-3.1/doc/hessian-2.0-spec.xtp

They are incompatible. :(

Will the community accepts the related implementation of the another version?
cc @zyfjeff

@zyfjeff
Copy link
Collaborator

zyfjeff commented Jun 4, 2024

These two are not the same thing, the former is the definition of a Hessian Web Service, and the latter is the serialization of the base type of Hessian

@wbpcode
Copy link
Collaborator Author

wbpcode commented Jun 4, 2024

Sorry, I pasted error link in the description. Updated. :)

@zyfjeff
Copy link
Collaborator

zyfjeff commented Jun 4, 2024

Obviously, sofa uses the draft version of the hessian2 spec, which is not officially supported, and we should not deviate from the official spec for a long time, sofa should be compatible with the official spec

@wbpcode
Copy link
Collaborator Author

wbpcode commented Jun 4, 2024

Ideally, yeah. But, I think it's hard (and maybe impossible?) for sofa to do this update. Because this require all sofa services in the cluster to update the codec lib version at same time. These is no any compatibility between these two versions. orz.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants