[CAMEL-13351] camel-netty4-http: error resolving relative path

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 2.23.2, 2.24.0, 2.22.4, 3.0.0-M2, 3.0.0
• Component/s: camel-netty4-http
• Estimated Complexity: Unknown
• Assignee: Luca Burgazzoli
• Reporter: Luca Burgazzoli
• Created: 20/Mar/19 12:33
• Updated: 23/Mar/19 09:02
• Resolved: 23/Mar/19 09:02

Description

DefaultNettyHttpBinding the uri used for request is computed as follow:

String uriForRequest = uri;
if (configuration.isUseRelativePath()) {
    int indexOfPath = uri.indexOf((new URI(uri)).getPath());
    if (indexOfPath > 0) {
        uriForRequest = uri.substring(indexOfPath);
    }
}

This has some issues:

• indexOf + substring are in fact the same as what we cna achieve with new URI(uri).getPath()
• in case the path is resolved to / then the logic is wrong as in case of an uri like http://something:80/ it will end up with //something:80/

Issue Links

• links to: GitHub Pull Request #35
• links to: GitHub Pull Request #2832