curl-library
[Patch] accept any RTSP session id
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
- application/octet-stream attachment: rtsp_sessionid_patch.diff