/**
 * Copyright 2013 Netflix, Inc.
 *
 * Licensed under the Apache License, Version 2.0 (the "License");
 * you may not use this file except in compliance with the License.
 * You may obtain a copy of the License at
 *
 *     http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */
package com.netflix.priam.cassandra.extensions;

import java.io.ByteArrayOutputStream;
import java.io.DataInputStream;
import java.io.FilterInputStream;
import java.net.HttpURLConnection;
import java.net.URL;

import com.google.common.base.Charsets;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;

public class DataFetcher
{
    private static final Logger logger = LoggerFactory.getLogger(DataFetcher.class);

    public static String fetchData(String url)
    {
        try
        {
            HttpURLConnection conn = (HttpURLConnection) new URL(url).openConnection();
            conn.setConnectTimeout(1000);
            conn.setReadTimeout(1000);
            conn.setRequestMethod("GET");
            if (conn.getResponseCode() != 200)
                throw new RuntimeException("Unable to get data for URL " + url);

            byte[] b = new byte[2048];
            ByteArrayOutputStream bos = new ByteArrayOutputStream();
            DataInputStream d = new DataInputStream((FilterInputStream) conn.getContent());
            int c = 0;
            while ((c = d.read(b, 0, b.length)) != -1)
                bos.write(b, 0, c);
            String return_ = new String(bos.toByteArray(), Charsets.UTF_8);
            logger.info(String.format("Calling URL API: %s returns: %s", url, return_));
            conn.disconnect();
            return return_;
        }
        catch (Exception ex)
        {
            throw new RuntimeException(ex);
        }
    }

}

@@ -1061,24 +1061,71 @@
  url)%0A    %7B%0A
+        DataInputStream responseStream = null;%0A
         try%0A
@@ -1292,32 +1292,33 @@
 ReadTimeout(1000
+0
 );%0A            c
@@ -1599,26 +1599,23 @@
     
-DataInput
+response
 Stream 
-d 
 = ne
@@ -1719,17 +1719,30 @@
 e ((c = 
-d
+responseStream
 .read(b,
@@ -2122,21 +2122,326 @@
 ;%0A        %7D%0A
+        finally%0A        %7B%0A            try%0A            %7B%0A                if(responseStream != null)%0A                    responseStream.close();%0A            %7D%0A            catch (Exception e)%0A            %7B%0A                logger.warn(%22Failed to close response stream from priam%22, e);%0A            %7D%0A        %7D%0A
     %7D%0A%0A%7D%0A

Increase read timeout and close response stream in DataFetcher. We had some problems with the short read timeout (previously 1 second), on the getSeeds call in particular, so just increasing the timeout value. Also, noticed the reponse stream was never directly closed, so now cleaning that up.