[rtmpdump] [PATCH] Better URL decoding support

[NhJm]

On Thu, Oct 25, 2012 at 12:19 AM, Steven Penny <svnpenn at gmail.com> wrote:

> On Wed, Oct 24, 2012 at 11:45 PM, NhJm wrote:
> > The _official_ RTMP client does not decode URLs prior to sending them to
> 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.)
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.mplayerhq.hu/pipermail/rtmpdump/attachments/20121025/f6b1bdb1/attachment.html>