RestEasy client: misuse of BasicClientConnManager: connection still allocated

Question

I am using the RestEasy client to retrieve a list of objects from a web server. This is my code:

@ApplicationScoped
public class RestHttpClient {
    private ResteasyClient client;

    @Inject
    private ObjectMapper mapper;

    @PostConstruct
    public void initialize() {
        HttpParams params = new BasicHttpParams();
        HttpConnectionParams.setConnectionTimeout(params, 5000);
        HttpConnectionParams.setSoTimeout(params, 5000);
        HttpClient httpClient = new DefaultHttpClient(params);
        this.client = new ResteasyClientBuilder().httpEngine(new ApacheHttpClient4Engine(httpClient)).build();
    }

    public <E> List<E> getList(final Class<E> resultClass, final String path,
            MultivaluedMap<String, Object> queryParams) {
        ResteasyWebTarget target = this.client.target(path);
        Response response = null;
        try {
            response = target.queryParams(queryParams).request().get();
            String jsonString = response.readEntity(String.class);
            TypeFactory typeFactory = TypeFactory.defaultInstance();
            List<E> list = this.mapper.readValue(
                    jsonString, typeFactory.constructCollectionType(ArrayList.class, resultClass));
            return list;
        } catch (Exception e) {
            // Handle exception
        } finally {
            if (response != null)
                response.close();
        }

        return null;
    }
}

, ... getList() , , " BasicClientConnManager: ". , 90% , . Response finally, , , , , . , , ? , , RestEasy. resteasy-client 3.0.4.Final.

Answer 1

, RestHttpClient, / .

ResteasyClientBuilder . , . , ResteasyClient ( " " ). , :

ResteasyClientBuilder clientBuilder = new ResteasyClientBuilder();
clientBuilder = clientBuilder.connectionPoolSize( 20 );
ResteasyWebTarget target = clientBuilder.build().target( "http://your.host" );

RestClientFactory . , ( ssl), .

public class RestClientFactory {

    public static class Options {
        private final String baseUri;
        private final String proxyHostname;
        private final String proxyPort;
        private final String keystore;
        private final String keystorePassword;
        private final String connectionPoolSize;
        private final String connectionTTL;

        public Options(String baseUri, String proxyHostname, String proxyPort) {
            this.baseUri = baseUri;
            this.proxyHostname = proxyHostname;
            this.proxyPort = proxyPort;
            this.connectionPoolSize = "100";
            this.connectionTTL = "500";
            this.keystore = System.getProperty( "javax.net.ssl.keyStore" );
            this.keystorePassword = System.getProperty( "javax.net.ssl.keyStorePassword" );
        }

        public Options(String baseUri, String proxyHostname, String proxyPort, String keystore, String keystorePassword, String connectionPoolSize, String connectionTTL) {
            this.baseUri = baseUri;
            this.proxyHostname = proxyHostname;
            this.proxyPort = proxyPort;
            this.connectionPoolSize = connectionPoolSize;
            this.connectionTTL = connectionTTL;
            this.keystore = keystore;
            this.keystorePassword = keystorePassword;
        }
    }

    private static Logger log = LoggerFactory.getLogger( RestClientFactory.class );

    public static <T> T createClient(Options options, Class<T> proxyInterface) throws Exception {
        log.info( "creating ClientBuilder using options {}", ReflectionToStringBuilder.toString( options ) );

        ResteasyClientBuilder clientBuilder = new ResteasyClientBuilder();

        ResteasyProviderFactory providerFactory = new ResteasyProviderFactory();
        RegisterBuiltin.register( providerFactory );
        providerFactory.getClientReaderInterceptorRegistry().registerSingleton( new ReaderInterceptor() {
            @Override
            public Object aroundReadFrom(ReaderInterceptorContext context) throws IOException, WebApplicationException {
                if (log.isDebugEnabled()) {
                    InputStream is = context.getInputStream();
                    String responseBody = IOUtils.toString( is );

                    log.debug( "received response:\n{}\n\n", responseBody );

                    context.setInputStream( new ByteArrayInputStream( responseBody.getBytes() ) );
                }
                return context.proceed();
            }
        } );
        clientBuilder.providerFactory( providerFactory );

        if (StringUtils.isNotBlank( options.proxyHostname ) && StringUtils.isNotBlank( options.proxyPort )) {
            clientBuilder = clientBuilder.defaultProxy( options.proxyHostname, Integer.parseInt( options.proxyPort ) );
        }

        // why the fuck do you have to specify the keystore with RestEasy?
        // not setting the keystore will result in not using the global one
        if ((StringUtils.isNotBlank( options.keystore )) && (StringUtils.isNotBlank( options.keystorePassword ))) {
            KeyStore ks;
            ks = KeyStore.getInstance( KeyStore.getDefaultType() );
            FileInputStream fis = new FileInputStream( options.keystore );
            ks.load( fis, options.keystorePassword.toCharArray() );
            fis.close();
            clientBuilder = clientBuilder.keyStore( ks, options.keystorePassword );
            // Not catching these exceptions on purpose
        }

        if (StringUtils.isNotBlank( options.connectionPoolSize )) {
            clientBuilder = clientBuilder.connectionPoolSize( Integer.parseInt( options.connectionPoolSize ) );
        }

        if (StringUtils.isNotBlank( options.connectionTTL )) {
            clientBuilder = clientBuilder.connectionTTL( Long.parseLong( options.connectionTTL ), TimeUnit.MILLISECONDS );
        }

        ResteasyWebTarget target = clientBuilder.build().target( options.baseUri );

        return target.proxy( proxyInterface );
    }
}

:

public interface SimpleClient
{
   @GET
   @Path("basic")
   @Produces("text/plain")
   String getBasic();
}

:

SimpleClient client = RestClientFactory.createClient( 
    new RestClientFactory.Options(
        "https://your.service.host",
        "proxyhost",
        "8080",
        "keystore.jks",
        "changeit",
        "20",
        "500"
    )
     ,SimpleClient.class
);

. : http://docs.jboss.org/resteasy/docs/3.0-beta-3/userguide/html/RESTEasy_Client_Framework.html#d4e2049