Re: [rtmpdump] [PATCH] Better URL decoding support

25 Oct 2012


      On Thu, Oct 25, 2012 at 12:19 AM, Steven Penny <svnpenn@gmail.com> wrote:
...
...
The _official_ RTMP client does not decode URLs prior to sending them to
On Wed, Oct 24, 2012 at 11:45 PM, NhJm wrote:
the
...
server, therefore rtmpdump should not be decoding URLs either.
So then what does this do? Taken from official master parseurl.c
if (*p == '%') {
                        unsigned int c;
                        sscanf(p+1, "%02x", &c);
                        *destptr++ = c;
                        pplen -= 3;
                        p += 3;
It does do urldecoding. It would probably be best to remove it. (And if
people need to include spaces in librtmp parameterized URLs, they can just
do it the hard way by using the parameters.)