Merge pull request #1670 from OpenRefine/issue1669

Simplify parsing of HTTP headers in fetch-url operation
This commit is contained in:
Owen Stephens 2018-07-11 15:45:23 +01:00 committed by GitHub
commit db13497744
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -338,16 +338,12 @@ public class ColumnAdditionByFetchingURLsOperation extends EngineDependentOperat
try { try {
URLConnection urlConnection = url.openConnection(); URLConnection urlConnection = url.openConnection();
if (_httpHeadersJson != null) { if (_httpHeadersJson != null) {
Map<String, String> httpHeaders = new HashMap<>();
for (int i = 0; i < _httpHeadersJson.length(); i++) { for (int i = 0; i < _httpHeadersJson.length(); i++) {
String headerLabel = _httpHeadersJson.getJSONObject(i).getString("name"); String headerLabel = _httpHeadersJson.getJSONObject(i).getString("name");
String headerValue = _httpHeadersJson.getJSONObject(i).getString("value"); String headerValue = _httpHeadersJson.getJSONObject(i).getString("value");
httpHeaders.put(headerLabel, headerValue); if (headerValue != null && !headerValue.isEmpty()) {
} urlConnection.setRequestProperty(headerLabel, headerValue);
for (String headerLabel : HttpHeadersSupport.getHttpHeaderLabels()) { }
HttpHeaderInfo info = HttpHeadersSupport.getHttpHeaderInfo(headerLabel);
urlConnection.setRequestProperty(info.header, httpHeaders.get(headerLabel));
} }
} }