Extraneous inclusion of logger in Base64Codec.php prevents overloading ESAPI about owasp-esapi-php

[GoogleCodeExporter]

Hi!

I cannot think of a need to bring custom code into ESAPI class. This class 
simply provides factory methods for security controls. It does not contain or 
is supposed to contain any "magic".
You are encouraged to write adapters for security controls that fit your needs 
which is working out of the box.


Some background on ESAPI for PHP: This is an explicit port of ESAPI for Java 
into PHP world and there you find also the inclusion and *use of the logger* in 
the Base64 codec.
@see 
http://code.google.com/p/owasp-esapi-java/source/browse/branches/1.4/src/main/ja
va/org/owasp/esapi/codecs/Base64.java

In ESAPI for PHP the logger is currently not used but this might change in 
future.

Original comment by [email protected] on 22 Sep 2010 at 7:34

[GoogleCodeExporter]

Hi there,

I don't intend for ESAPI.php to be overridden like that. We're using factories 
to create the non-static classes, so I'd like for you to consider writing new 
or overriding the reference implementation, and instantiating that. ESAPI.php 
is a short cut to getting all the necessary code in place. 

Decoupling features is an interesting issue, as realistically, I think we 
should be saying the Logger and IDS and Exception classes will always be 
included by any class as they are a way of ensuring consistent ESAPI behavior 
and code re-use. 

However, I do know that over riding the ESAPI base / reference classes has 
never been done before in a real non-test app, so I will check that this 
actually works before we release 1.0. 

Original comment by [email protected] on 17 Jun 2011 at 3:34

• Added labels: Maintainability, Milestone-Release1.0

[GoogleCodeExporter]

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 [email protected] on 17 Jun 2011 at 3:36