Uploaded image for project: 'Pentaho Data Integration - Kettle'
  1. Pentaho Data Integration - Kettle
  2. PDI-13357

REST uses a wrong http client: unable to use custom trust store

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Open
    • Severity: High
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: Backlog
    • Component/s: Step
    • Labels:
    • Story Points:
      0
    • PDI Sub-component:
    • Notice:
      When an issue is open, the "Fix Version/s" field conveys a target, not necessarily a commitment. When an issue is closed, the "Fix Version/s" field conveys the version that the issue was fixed in.

      Description

      ApacheHttpClient unable to handle HTTPSProperties.PROPERTY_HTTPS_PROPERTIES therefore it is unable to use custom provided trust store (with self signed certificates for instance)

      the same case: https://java.net/projects/jersey/lists/users/archive/2012-06/message/14

      patch:

      Index: engine/src/org/pentaho/di/trans/steps/rest/Rest.java
      IDEA additional info:
      Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
      <+>UTF-8
      ===================================================================
      --- engine/src/org/pentaho/di/trans/steps/rest/Rest.java	(revision 6edae5df20ffa33bafad4f47260646a49c7a9ee8)
      +++ engine/src/org/pentaho/di/trans/steps/rest/Rest.java	(revision )
      @@ -58,7 +58,6 @@
       import com.sun.jersey.api.client.UniformInterfaceException;
       import com.sun.jersey.api.client.WebResource;
       import com.sun.jersey.api.client.filter.HTTPBasicAuthFilter;
      -import com.sun.jersey.client.apache.ApacheHttpClient;
       import com.sun.jersey.client.apache.config.ApacheHttpClientConfig;
       import com.sun.jersey.client.apache.config.DefaultApacheHttpClientConfig;
       import com.sun.jersey.client.urlconnection.HTTPSProperties;
      @@ -241,7 +240,7 @@
       
         private Client getClient() {
       
      -    Client c = ApacheHttpClient.create( data.config );
      +    Client c = Client.create( data.config );
           if ( data.basicAuthentication != null ) {
             c.addFilter( data.basicAuthentication );
           }
      

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              Unassigned Unassigned
              Reporter:
              vladimir.dolzhenko Vladimir Dolzhenko
              Votes:
              4 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated: