kitpages/kitpagesfilesystembundle

This is a bundle that provides a filesystem abstraction layer. Basicaly this does the same as the gaufrette library from KnpLabs, but it manages much more efficiently big files. We can manage a 2GB file with a memory limit of 128Mo for the PHP process. We never transfert the entire content in a $content variable.

kitpagesfilesystembundle's Issues

Insights Code scanning report

Hi

Thank you for your great work with this bundle and your contribution to the open source community.

I have put the bundle through Sensiolabs Insights code scanning and just wanted to share the findings with you.

The scanning found 11 Major, 20 Minor, 36 Info issues/recommendations.

They are:
PHP configuration should not be changed dynamically
MajorBugrisk
in Service/Adapter/AmazonS3.php, line 208
ini_set('zlib.output_compression', 'Off');
Changing PHP configuration dynamically through ini_set() may create hard to debug errors.

See https://insight.sensiolabs.com/what-we-analyse/php.dynamically_change_configuration

---

exit() and die() functions should be avoided
MajorBugrisk
in Service/Adapter/AmazonS3.php, line 260

See https://insight.sensiolabs.com/what-we-analyse/php.use_exit_function

---

PHP response functions should not be used 6
MajorBugrisk
in Service/Adapter/AmazonS3.php, line 230

header('Cache-Control: public, max-age=0');
Using PHP response functions (like header() here) is discouraged, as it bypasses the Symfony event system. Use the HttpFoundationResponse class instead.

in Service/Adapter/AmazonS3.php, line 231
in Service/Adapter/AmazonS3.php, line 232
in Service/Adapter/AmazonS3.php, line 233
in Service/Adapter/AmazonS3.php, line 234
in Service/Adapter/AmazonS3.php, line 236

See https://insight.sensiolabs.com/what-we-analyse/symfony.use_php_response_function

---

Absolute path constants DIR and FILE should not be used 3
MajorBugrisk
in Test/AbstractAdapterTest.php, line 34
if (!is_dir(dirname(FILE).'/../Tests/tmp')) {
DIR and FILE constants may conflict with the Symfony resource overriding system

in Test/AbstractAdapterTest.php, line 35
in Test/AbstractAdapterTest.php, line 39

See https://insight.sensiolabs.com/what-we-analyse/symfony.dependency_injection.use_dir_file_constant

Attached is a more detailed report.
KitpagesFileSystemBundle - SensioLabsInsight.pdf

If possible for you to fix these issues/comments/recommendations with a future commit, it will be appreciated.

Kind Regards
Renier Lewis

AWS SDK in Composer

If aws-sdk-for-php is an optional package (as is suggested in your documentation for deps files) could it be listed in composer.json as a suggestion rather than a requirement please?

e.g.

"require": {
    "php": ">=5.3.2",
    "symfony/framework-bundle": "2.0.*",
    "kitpages/util-bundle": ">=v1.0.0"
},
"suggest": {
    "aws-sdk-for-php": "Add for AWS integration"
}

add autoload modification in readme

add the line we have in autoload.php to add in the readme file