views:

397

answers:

2

The following function will pull down first X messages from Twitter firehose, but looks like WebResponse blocks and never exits the function:

public void GetStatusesFromStream(string username, string password, int nMessageCount)
{
    WebRequest request = WebRequest.Create("http://stream.twitter.com/1/statuses/sample.json");
    request.Credentials = new NetworkCredential(username, password);

    using (WebResponse response = request.GetResponse())
    {
        using (var stream = response.GetResponseStream())
        {
            using (var reader = new StreamReader(stream))
            {
                while (!reader.EndOfStream)
                {
                    Console.WriteLine(reader.ReadLine());

                    if (nMessageCount-- < 0)
                        break;
                }
                Console.WriteLine("Start iDispose");
            }
            Console.WriteLine("Never gets here!!!");
        }
    }

   Console.WriteLine("Done - press a key to exit");
   Console.ReadLine();
}

But the following works fine:

public void GetStatusesFromStreamOK(string username, string password, int nMessageCount)
    {
    byte[] encbuff = System.Text.Encoding.UTF8.GetBytes(username + ":" + password);
    //request.Headers.Add("Authorization", "Basic " + Convert.ToBase64String(encbuff));

    string requestString = "GET /1/statuses/sample.json HTTP/1.1\r\n";
    requestString += "Authorization: " + "Basic " + Convert.ToBase64String(encbuff) + "\r\n";
    requestString += "Host: stream.twitter.com\r\n";
    requestString += "Connection: keep-alive\r\n";
    requestString += "\r\n";

    using (TcpClient client = new TcpClient())
    {
        client.Connect("stream.twitter.com", 80);

        using (NetworkStream stream = client.GetStream())
        {
            // Send the request.
            StreamWriter writer = new StreamWriter(stream);
            writer.Write(requestString);
            writer.Flush();

            // Process the response.
            StreamReader rdr = new StreamReader(stream);

            while (!rdr.EndOfStream)
            {
                Console.WriteLine(rdr.ReadLine());
                if (nMessageCount-- < 0)
                    break;
            }

        }
    }

   Console.WriteLine("Done - press a key to exit");
   Console.ReadLine();

}

What am I doing wrong?

A: 

Looks like it has something to do with the disposal process or "using"...

The following code works fine (no "using" statements):

public static void GetStatusesFromStream(string username, string password, int nMessageCount)
    {
        WebRequest request = WebRequest.Create("http://stream.twitter.com/1/statuses/sample.json");
        request.Credentials = new NetworkCredential(username, password);

        WebResponse response = request.GetResponse();
        {
            var stream = response.GetResponseStream();
            {
                var reader = new StreamReader(stream);
                {
                    while (!reader.EndOfStream)
                    {
                        Console.WriteLine(reader.ReadLine());

                        if (nMessageCount-- < 0)
                            break;
                    }
                }
                Console.WriteLine("Never gets here!!!");
            }
        }

        Console.WriteLine("Done - press a key to exit");
        Console.ReadLine();
    }
drakaan
Agree it gets further, but only because you're delaying the .NET garbage collection. If you run the function a couple of times the garbage collection will kick in and the program will hang as before.
martimedia
+1  A: 

Cast your WebRequest as an HttpWebRequest, then before the break call request.Abort()

Chris Haas
Thanks, adding request.Abort() did the trick, regardless of casting the WebRequest as a HttpWebRequest.Not sure why the original code worked for some, but would strongly advise people to add request.Abort to their code if they want to run on a wide range of .NET clients.
martimedia