On Fri, 2010-01-08 at 23:59 +0100, Daniel Stenberg wrote:
> On Fri, 8 Jan 2010, Chris Conroy wrote:
>
> >> Cool, I'm looking forward to that! Are you in sync with CVS or are you
> >> working against a fixed release?
>
> > Right now, I'm working against 7.19.7, but I could pretty easily upgrade to
> > the latest CVS if that is your preference.
>
> I'm just thinking that the upcoming release is probably one of the larger
> changes in the source code since a long time so chances are fairly high that a
> patch against 7.19.7 won't apply cleanly on CVS/7.20.0.
Will do for the final patch. I'm including a "preview" patch in case
anyone has any time this weekend to look it over and provide feedback so
I can hit the ground running on Monday. I'll provide a patch against CVS
early next week.

>
> So yes, when you do present a patch for this I'd prefer it to be against CVS
> to make it easier for me.
> > Well, that's good news because it means we know the source of the issue.
> > However, letting it read all incoming data is almost definitely not what we
> > want to happen here. Once the server starts sending interleaved RTP data, it
> > will do so until the stream is over. However, the application will need the
> > ability to issue other commands over the same connection (e.g. PAUSE) while
> > this is happening. Let me know if you have any thoughts on how to work
> > around this problem.
>
> I'll confess I don't know RTSP so I may sound stupid here, but why exactly are
> you "leaving" the connection as done when a transfer is clearly still going?
Basically RTSP allows for RTP (stream data) to be interleaved on the
same connection. It's best to think of it as a separate transfer that is
using the same connection. I believe I have a suitable workaround which
you'll see in the patch. Essentially if it's RTSP I run
Curl_getconnectinfo() to do a PEEK onto the stream in order to
distinguish between data ready and a close. Since this message thread is
about the connection close issue, I'll start a separate thread to
discuss the patch.

--Chris

-------------------------------------------------------------------
List admin: http://cool.haxx.se/list/listinfo/curl-library
Etiquette: http://curl.haxx.se/mail/etiquette.html
Received on 2010-01-09