cURL / Mailing Lists / curl-library / Single Mail

curl-library

[Patch] accept any RTSP session id

From: Erik Janssen <erik.janssen_at_axis.com>
Date: Tue, 9 Aug 2016 15:56:47 +0000

Hi,
I propose the following small change to make libcurl work in communication with gstreamer-based RTSP servers. See attached patch against 7.50.1.
The original code validates the session id to be in accordance with the RFC. I think it is better not to do that:

- For curl the actual content is a don't care.

- The clarity of the RFC is debatable, is $ allowed or only as \$, that is imho not clear

- Gstreamer seems to url-encode the session id but % is not allowed by the RFC
With this patch curl will correctly handle real-life lines like:
Session: biTN4Kc.8%2B1w-AF.; timeout=60
And code reduction :)
Interested to hear your opinion.

Regards,

Erik

-------------------------------------------------------------------
List admin: https://cool.haxx.se/list/listinfo/curl-library
Etiquette: https://curl.haxx.se/mail/etiquette.html

Received on 2016-08-09