[CAMEL-10640] Custom AsyncHttpClientConfig not used in WsEndpoint

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.18.1
• Fix Version/s: 2.18.2, 2.19.0
• Component/s: camel-ahc-ws
• Patch Info: Patch Available
• Estimated Complexity: Novice
• Assignee: Andrea Cosentino
• Reporter: Thomas Günter
• Created: 22/Dec/16 10:28
• Updated: 22/Dec/16 13:09
• Resolved: 22/Dec/16 13:02

Description

Provided custom AsyncHttpClientConfig is not used by WsEndpoint.java.

Current implementation:

WsEndpoint.java:109

    protected AsyncHttpClient createClient(AsyncHttpClientConfig config) {
        AsyncHttpClient client;
        if (config == null) {
            config = new DefaultAsyncHttpClientConfig.Builder().build();
            client = new DefaultAsyncHttpClient(config);
        } else {
            client = new DefaultAsyncHttpClient();
        }
        return client;
    }

Expected implementation:

WsEndpoint.java:109

    protected AsyncHttpClient createClient(AsyncHttpClientConfig config) {
        AsyncHttpClient client;
        if (config == null) {
            config = new DefaultAsyncHttpClientConfig.Builder().build();
            client = new DefaultAsyncHttpClient(config);
        } else {
            client = new DefaultAsyncHttpClient(config);
        }
        return client;
    }

Issue Links

• links to: GitHub Pull Request #1365
• links to: Pull Request 1365