close HttpClientAdapter's reader instead of InputStream
when getting http entity as string we didn't close
the top-most level object. this was detected by findbugs.
diff --git a/src/com/google/enterprise/adaptor/HttpClientAdapter.java b/src/com/google/enterprise/adaptor/HttpClientAdapter.java
index a317633..2b56d3e 100644
--- a/src/com/google/enterprise/adaptor/HttpClientAdapter.java
+++ b/src/com/google/enterprise/adaptor/HttpClientAdapter.java
@@ -240,15 +240,15 @@
}
InputStream is = getResponseEntityAsStream();
StringBuilder sb = new StringBuilder();
+ Reader reader = new InputStreamReader(is, charset);
try {
- Reader reader = new InputStreamReader(is, charset);
char[] buf = new char[1024];
int read;
while ((read = reader.read(buf)) != -1) {
sb.append(buf, 0, read);
}
} finally {
- is.close();
+ reader.close();
}
return sb.toString();
}
