You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@cxf.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2017/01/11 12:42:58 UTC
[jira] [Commented] (CXF-7210) StringIndexOutOfBoundsException
during construction of failover request
[ https://issues.apache.org/jira/browse/CXF-7210?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15818203#comment-15818203 ]
ASF GitHub Bot commented on CXF-7210:
-------------------------------------
GitHub user sebhoss opened a pull request:
https://github.com/apache/cxf/pull/221
[CXF-7210] Fix StringIndexOutOfBoundsException
The logic goes as follows:
1) Base- & request-URI are the same, then only add base-URI.
2) If request-URI starts with base-URI, only add the parts from request-URI that are not already included in base-URI
3) Otherwise add both base-URI and request-URI
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/sebhoss/cxf master
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/cxf/pull/221.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 #221
----
commit 1a8ae7afb254b77118aba543df2a720547a3ccf2
Author: Sebastian Hoß <ma...@shoss.de>
Date: 2017-01-11T12:39:31Z
[CXF-7210] Fix StringIndexOutOfBoundsException
Signed-off-by: Sebastian Hoß <ma...@shoss.de>
----
> StringIndexOutOfBoundsException during construction of failover request
> -----------------------------------------------------------------------
>
> Key: CXF-7210
> URL: https://issues.apache.org/jira/browse/CXF-7210
> Project: CXF
> Issue Type: Bug
> Components: JAX-RS
> Reporter: Sebastian Hoß
> Priority: Trivial
>
> In {{AbstractClient}} we [have the following|https://github.com/apache/cxf/blob/master/rt/rs/client/src/main/java/org/apache/cxf/jaxrs/client/AbstractClient.java#L650]:
> {code}
> UriBuilder builder = new UriBuilderImpl().uri(newBaseURI);
> String basePath = reqURIPath.startsWith(baseURIPath) ? baseURIPath : getBaseURI().getRawPath();
> builder.path(reqURIPath.equals(basePath) ? "" : reqURIPath.substring(basePath.length()));
> {code}
> This covers two out of three potential cases:
> 1) {{baseURIPath}} and {{reqURIPath}} have the same value which will result in only the base path being added to the builder.
> 2) {{baseURIPath}} and {{reqURIPath}} have a different value in which case the builder will get the base path and and everything from the request path that is not already included in the base path.
> What's missing (and what is causing the exception below) is that the base path might be longer than the request path in which case {{reqURIPath.substring(basePath.length())}} throws the following stack trace:
> {code}
> Caused by: java.lang.StringIndexOutOfBoundsException: String index out of range: -7
> at java.lang.String.substring(String.java:1875)
> at org.apache.cxf.jaxrs.client.AbstractClient.calculateNewRequestURI(AbstractClient.java:652)
> at org.apache.cxf.jaxrs.client.AbstractClient.calculateNewRequestURI(AbstractClient.java:642)
> at org.apache.cxf.jaxrs.client.AbstractClient.retryInvoke(AbstractClient.java:688)
> at org.apache.cxf.jaxrs.client.AbstractClient$RetryableImpl.invoke(AbstractClient.java:1174)
> at org.apache.cxf.clustering.FailoverTargetSelector.performFailover(FailoverTargetSelector.java:207)
> at org.apache.cxf.clustering.FailoverTargetSelector.complete(FailoverTargetSelector.java:166)
> at org.talend.esb.servicelocator.cxf.internal.LocatorTargetSelector.complete(LocatorTargetSelector.java:57)
> at org.apache.cxf.jaxrs.client.AbstractClient.preProcessResult(AbstractClient.java:575)
> {code}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)