[Bug 742869] Remove URI-escaping of RTSP session-id

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=742869
  GStreamer | gst-rtsp-server | git

Sebastian Dröge (slomo) <slomo> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
                 CC|                            |slomo at coaxion.net
         Resolution|                            |FIXED
   Target Milestone|HEAD                        |1.5.1

--- Comment #2 from Sebastian Dröge (slomo) <slomo at coaxion.net> 2015-01-14 17:44:09 UTC ---
Did you file a bug against ffmpeg/libav about this?

commit 94f3e18c5bb2d8a054ab3b3d4828b9c77fd7f398
Author: Sebastian Rasmussen <sebrn at axis.com>
Date:   Tue Jan 13 15:29:29 2015 +0100

    Revert "rtsp-session-pool: Make sure session IDs are properly URI-escaped"

    This reverts commit 935e8f852d050b4939f1d0f44b38e9b55a2fbe36.

    RFC 2326 states that session IDs may consist of alphanumeric as well as
    the safe characters $-_.+ -- N.B. the percent character is not allowed.

    Previously the session ID was URI-escaped, this meant that any character
    which was not alphanumeric or any of the characters +-._~ would be
    percent encoded. While the RFC (surprisingly) mentions that linear white
    space in session IDs should be URI-escaped, it does not say anything
    about other characters. Moreover no white space is allowed in the
    session ID. Finally the percent character which is the result of
    URI-escaping is not allowed in a session ID.

    So there is no reason to do any URI-escaping, and now it is removed.

    https://bugzilla.gnome.org/show_bug.cgi?id=742869

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.