search

mchibouni / owasp-esapi-php

Automatically exported from code.google.com/p/owasp-esapi-php
Other
0 stars 0 forks source link

BaseValidationRule missing sanitize method #8

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago 
Please add a sanitize method to the BaseValidationRule

Original issue reported on code.google.com by mike.bob...@gmail.com on 9 Oct 2009 at 7:34

GoogleCodeExporter commented 9 years ago

Original comment by mike.bob...@gmail.com on 9 Oct 2009 at 7:41

GoogleCodeExporter commented 9 years ago 
De-allocating all defects to me.

Original comment by vande...@gmail.com on 14 Jul 2010 at 9:27

GoogleCodeExporter commented 9 years ago 
I went ahead and took the simplest route here, but I've added that method. It 
currently throws a ValidationAvailabilityException (my best guess) unless 
overridden with a real implementation in a child class. I felt that was safer 
than trying to guess what a default "safe" value would be for all possible 
implementations.

I also commented out the global $ESAPI line from the constructor because it 
wasn't used in the constructor or the majority of BaseValidationRule's child 
classes. The 3 that did use it also called it individually.

Original comment by Fen...@gmail.com on 22 Jul 2010 at 4:15

Attachments:

GoogleCodeExporter commented 9 years ago 
All these issues must be dealt with before 1.0. New issues will be on a case by 
case basis as to whether we hold up 1.0 or not.

Original comment by vande...@gmail.com on 17 Jun 2011 at 3:36