[Bug-wget] Bug in processing url query arguments that have '/'

2011-08-07 Thread Peng Yu
Hi, The following line is in utils.c. # in acceptable (const char *s) while (l s[l] != '/') --l; if (s[l] == '/') s += (l + 1); It essentially gets a substring after the last '/'. However, when a query has '/', this is problematic. For example, the above code snip will extract

Re: [Bug-wget] Bug in processing url query arguments that have '/'

2011-08-07 Thread Giuseppe Scrivano
Hello Peng, AFAICS, `s' is a path, so '/' in the query string is escaped and `acceptable' doesn't see it. As for your example: http://xxx.org/somescript?arg1=/xxy `s' in this case will be something like: xxx.org/somescript?arg1=%2Fxxy Do you have any example where it doesn't work? Cheers,

Re: [Bug-wget] Bug in processing url query arguments that have '/'

2011-08-07 Thread Peng Yu
On Sun, Aug 7, 2011 at 6:14 AM, Giuseppe Scrivano gscriv...@gnu.org wrote: Hello Peng, AFAICS, `s' is a path, so '/' in the query string is escaped and `acceptable' doesn't see it. As for your example: http://xxx.org/somescript?arg1=/xxy `s' in this case will be something like:

[Bug-wget] 'or' and flexible expression of wget options

2011-08-07 Thread Peng Yu
Hi, It seems that all the options checked in download_child_p() are AND'ed. In gnu find, options can be and'ed or or'ed much more flexibly. I looked at wget source, it is not clear to me that flexible expression can be supported by the current wget cmdline option parsing framework. Does anybody