Taras,

On Sat, Jun 9, 2012 at 9:39 AM, Taras <ox...@oxdef.info> wrote:
> Andres,
>
> in my branch I have added analyzeResponseBody user configurable
> parameter into globalRedirect plugin [0]. My point is that usually check
> for 301/302 redirect is enough to detect globalredirect and we don't
> really need to analyze with regex response body for meta-tags and
> JavaScript (regex is always slow gear). But for **backward
> compatibility** default value for this option is **True**. So from
> default behavior of w3af here will remain the same. Merge with the trunk?

    It seems that I'm always against your ideas, but I just need to
understand things and that's why I always ask: "Why?" Have you noticed
that this particular regular expression / plugin was slow? Is this a
performance improvement or is there something else that we need to
know? If its just because of the performance improvement, I would
recommend changing from the regular expression that matches the
scripts:

    self._scriptre = re.compile('< *script.*?>(.*)< */ *script *>',
re.IGNORECASE | re.DOTALL )

    To response.getDOM() and then applying some XPATH to the dom
object in order to find/retrieve the script text. Another way would be
to rewrite the regular expression (maybe add a ? after the "< *" in
order to speed it up a little bit.

    My point is that if the problem is performance, we better fix it
instead of disabling the part that brings us problems. Also, users
disabling this would miss some vulnerabilities without knowing why
(unless they read the code).

Regards,

> [0]
> http://w3af.svn.sourceforge.net/viewvc/w3af/branches/oxdef/plugins/audit/globalRedirect.py?revision=5083&view=markup
> --
> Taras
> http://oxdef.info
>
> ------------------------------------------------------------------------------
> Live Security Virtual Conference
> Exclusive live event will cover all the ways today's security and
> threat landscape has changed and how IT managers can respond. Discussions
> will include endpoint security, mobile security and the latest in malware
> threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
> _______________________________________________
> W3af-develop mailing list
> W3af-develop@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/w3af-develop



-- 
Andrés Riancho
Project Leader at w3af - http://w3af.org/
Web Application Attack and Audit Framework
Twitter: @w3af
GPG: 0x93C344F3

------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
W3af-develop mailing list
W3af-develop@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/w3af-develop

Reply via email to