Uploaded image for project: 'Camel'
  1. Camel
  2. CAMEL-10640

Custom AsyncHttpClientConfig not used in WsEndpoint

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • 2.18.1
    • 2.18.2, 2.19.0
    • camel-ahc-ws
    • None
    • Patch Available
    • Novice

    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;
          }
      

      Attachments

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            acosentino Andrea Cosentino
            tom_guenter Thomas Günter
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment