[PEPr] Comment on Caching::phpfastcache
"Michael Gauthier" <[email protected]> Mon, 13 May 2013 22:06:50 +0100 (BST)
| Newsgroups | gmane.comp.php.pear.devel |
|---|---|
| Message-ID | <[email protected]> |
After looking at the code, I agree with Bertand. This class is doing too much. For example, emailing error messages does not belong in a cache library. There is definitely room in PEAR for a cache library, but the code needs to be split into multiple reusable classes rather than one monolithic static class. Serializing cache data is potentially a security issue as a vulnerability in any backend now has the potential to execute code on the server. -- http://pear.php.net/pepr/pepr-proposal-show.php?id=692 -- PEAR Development Mailing List (http://pear.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php