[
https://issues.apache.org/jira/browse/CXF-6923?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15306730#comment-15306730
]
ASF GitHub Bot commented on CXF-6923:
-------------------------------------
GitHub user grgrzybek opened a pull request:
https://github.com/apache/cxf/pull/140
[CXF-6923] Handle org.omg.CORBA.SystemException for Jacorb correctly
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/grgrzybek/cxf CXF-6923-3.1.x
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/cxf/pull/140.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #140
----
commit 0759f57a0786ef71a25e03c4d65c58632be3a18a
Author: Grzegorz Grzybek <gr.grzy...@gmail.com>
Date: 2016-05-30T14:49:06Z
[CXF-6923] Handle org.omg.CORBA.SystemException for Jacorb correctly
----
> org.omg.CORBA.TIMEOUT is not handled with Jacorb implementation
> ---------------------------------------------------------------
>
> Key: CXF-6923
> URL: https://issues.apache.org/jira/browse/CXF-6923
> Project: CXF
> Issue Type: Bug
> Components: CORBA Binding
> Affects Versions: 3.1.6
> Reporter: Grzegorz Grzybek
>
> When using Jacorb as ORB implementation at client side:
> {code:java}
> System.setProperty("org.omg.CORBA.ORBClass", "org.jacorb.orb.ORB");
> System.setProperty("org.omg.CORBA.ORBSingletonClass",
> "org.jacorb.orb.ORBSingleton");
> {code}
> we can set {{jacorb.connection.client.pending_reply_timeout}} system property
> that configures client-side to throw {{org.omg.CORBA.TIMEOUT}} when timeout
> occurs while waiting for reply.
> The problem is different implementation of DII request code:
> {code:java|title=com.sun.corba.se.impl.corba.RequestImpl#doInvocation()}
> } catch( SystemException ex ) {
> _env.exception(ex);
> // NOTE: The exception should not be thrown.
> // However, JDK 1.4 and earlier threw the exception,
> // so we keep the behavior to be compatible.
> throw ex;
> {code}
> {code:java|title=org.jacorb.orb.dii.Request#_invoke()}
> catch (Exception e)
> {
> if (logger.isDebugEnabled ())
> {
> logger.debug("DII Request caught Exception", e);
> }
> env.exception (e);
> break;
> }
> {code}
> PR + test soon
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
