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

Attachment: rtsp_sessionid_patch.diff
Description: rtsp_sessionid_patch.diff

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

Reply via email to