From 92ab39d4f3682f24635c1a3678e4534a5114e6af Mon Sep 17 00:00:00 2001 From: guange <8863824@gmail.com> Date: Mon, 20 Feb 2017 12:05:31 +0800 Subject: [PATCH] httpclient didn't release when http validateResponse false. --- .gitignore | 1 + .../java/com/offbytwo/jenkins/client/JenkinsHttpClient.java | 4 ++-- 2 files changed, 3 insertions(+), 2 deletions(-) mode change 100644 => 100755 jenkins-client/src/main/java/com/offbytwo/jenkins/client/JenkinsHttpClient.java diff --git a/.gitignore b/.gitignore index e73a9e03..9dc43422 100644 --- a/.gitignore +++ b/.gitignore @@ -5,3 +5,4 @@ target .project .settings test-output +.DS_Store diff --git a/jenkins-client/src/main/java/com/offbytwo/jenkins/client/JenkinsHttpClient.java b/jenkins-client/src/main/java/com/offbytwo/jenkins/client/JenkinsHttpClient.java old mode 100644 new mode 100755 index df58360f..ee4342bd --- a/jenkins-client/src/main/java/com/offbytwo/jenkins/client/JenkinsHttpClient.java +++ b/jenkins-client/src/main/java/com/offbytwo/jenkins/client/JenkinsHttpClient.java @@ -339,8 +339,8 @@ public String post_xml(String path, String xml_data, boolean crumbFlag) throws I } HttpResponse response = client.execute(request, localContext); getJenkinsVersionFromHeader(response); - httpResponseValidator.validateResponse(response); try { + httpResponseValidator.validateResponse(response); return IOUtils.toString(response.getEntity().getContent()); } finally { EntityUtils.consume(response.getEntity()); @@ -386,8 +386,8 @@ public String post_text(String path, String textData, ContentType contentType, b } HttpResponse response = client.execute(request, localContext); getJenkinsVersionFromHeader(response); - httpResponseValidator.validateResponse(response); try { + httpResponseValidator.validateResponse(response); return IOUtils.toString(response.getEntity().getContent()); } finally { EntityUtils.consume(response.getEntity());