search

klaubert / waf-fle

WAF-FLE, ModSecurity Console
http://waf-fle.org
GNU General Public License v2.0
140 stars 74 forks source link

Better url path and query handling #10

Closed sakalajuraj closed 10 years ago

sakalajuraj commented 10 years ago

parse_url php function handles url path and query better when protocol://host section is present. Adding dummy protocol://host section improves url parsing for example when there is another URL in the query. For example /?path=http://thaitacticalgear.com//live/wp-includes/images/t.gif??' is not parsed correctly without this change. This change solves the issue 16 at http://code.google.com/p/waf-fle/issues/

klaubert commented 10 years ago

I had tried with your example (and variations), but I still not able to reproduce what you have, this can be some issue with PHP version. However, I see no problem in merge your code, once it don't saw any collateral effect.

Thanks for contribute.

Klaubert