-
Notifications
You must be signed in to change notification settings - Fork 38.4k
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
Support Hessian 4.0.1 with Spring 3.0 [SPR-6041] #10710
Comments
Peter Dettman commented Patch against r1748. Requires the Hessian dependency to be updated to 4.0.1. |
Juergen Hoeller commented The problem here is that Resin 3.x (the server) keeps shipping with Hessian 3.x. We require 3.2 or higher there now for Spring 3.0 but I'm afraid we can't raise this any higher, since Resin cannot be upgraded to a new Hessian version without upgrading the entire server (as far as I'm aware). I expect Caucho to fix any critical bugs in Hessian 3.x as well, or at least in Hessian as shipped with Resin 3.x. After all, many people are on Resin 3.x now and will likely stay there for a while. Are there any issues with Spring 3.0 M4's Hessian support against Hessian 4.0.1, actually? In other words, is there any need for using the new Hessian 4 API? It seems to work fine with the old API (i.e. Spring 3.0 M4's code) in my tests... in which case we could still support Hessian 3.2, while recommending the use of Hessian 4.0 or higher with Spring 3.0. Juergen |
Yanming Zhou commented hessian-4.0.1 doesn't works with spring2.5.6 neither Exception in thread "main" org.springframework.remoting.RemoteAccessException: Cannot access Hessian remote service at http://localhost:8080/remoting/userManager; nested exception is com.caucho.hessian.io.HessianProtocolException: '�' is an unknown code |
Greg Ferguson commented Juergen, what version of Hessian (from http://hessian.caucho.com/index.xtp#Java) does the
Thanks in advance. |
Juergen Hoeller commented Greg, as far as my own tests went back in the Spring 3.0 RC phase, things worked just fine with Hessian 4.0.1 - so I assume Spring 3.0 GA will work with Hessian 4.0.2 as well. We're going to re-test this for Spring 3.0.1 and officially declared Hessian 4.0.x to be supported then. With respect to Hessian 3.1.6, while it might work, we are not officially supporting it anymore. Spring 3.0 requires Hessian 3.2 or above. Juergen |
Greg Ferguson commented Thanks...that's exactly what I needed to know. |
Juergen Hoeller commented We verified that Hessian 4.0.3 works fine with Spring 3.0.1. The Hessian2 protocol (to be activated through HessianProxyFactoryBean's "hessian2" flag) seems to work fine as well now, in contrast to the issues that we had with Hessian 3.2.1 back then. From that perspective, I'd recommend using Hessian 4.0.x now, even if we need to preserve compatibility with Hessian 3.2.x as well. Juergen |
Peter Dettman opened SPR-6041 and commented
The Caucho hessian lib has now stabilised its v2.0 protocol and rounded up several regressions, with the release of v4.0.1 (http://caucho.com/download/hessian-4.0.1.jar).
In 4.0.1, the HessianSkeleton class provides a ready-made invoke method that Spring's HessianExporter can call (instead of duplicating its code). I am attaching a patch that modifies HessianExporter.java to do so. The debugging output part of the code is still duplicated of necessity until some suitable refactoring of HessianSkeleton is done.
As far as backward compatibility is concerned, I believe users should avoid using Hessian 2.0 protocol from previous versions (that may be an overly strict interpretation). Hessian 1.0 protocol should work fine with a v4.0.1 server. I understand that there could be problems if people are using HessianOutput/Hessian2Output or streaming versions directly.
I recommend Caucho's Hessian 4.0.1 be used for Spring 3.0, a fresh start of sorts.
See also:
http://maillist.caucho.com/pipermail/hessian-interest/2009-June/000750.html (and surrounding discussion on that list)
http://jira.springframework.org/browse/SPR-5469
http://bugs.caucho.com/view.php?id=3646
Affects: 3.0 M4
Attachments:
Issue Links:
2 votes, 4 watchers
The text was updated successfully, but these errors were encountered: