Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Maximum execution time of 120 seconds exceeded #20

Open
tornadofay opened this issue Jan 29, 2019 · 2 comments
Open

Maximum execution time of 120 seconds exceeded #20

tornadofay opened this issue Jan 29, 2019 · 2 comments

Comments

@tornadofay
Copy link

first of all thanks for this package, it's really great package.

I'm getting the following error when I go to http://myapp/decompose

Maximum execution time of 120 seconds exceeded {"userId":1,"email":"******@gmail.com","exception":"[object] (Symfony\\Component\\Debug\\Exception\\FatalErrorException(code: 1): Maximum execution time of 120 seconds exceeded at C:\\wamp\\www\\SPF\\vendor\\lubusin\\laravel-decomposer\\src\\Decomposer.php:269)

so I followed the error to \lubusin\laravel-decomposer\src\Decomposer.php

private static function folderSize($dir)
    {
        $size = 0;
        foreach (glob(rtrim($dir, '/').'/*', GLOB_NOSORT) as $each) {
            $size += is_file($each) ? filesize($each) : self::folderSize($each);
        }
        return $size;
    }

I simply commented this part which calculate application size

// foreach (glob(rtrim($dir, '/').'/*', GLOB_NOSORT) as $each) {
//     $size += is_file($each) ? filesize($each) : self::folderSize($each);
// }

it gave me App Size: 0 B but everything else worked fine. so I don't know if this problem on my dev machine only, but will it run fine on production server ? or not.

Thanks.

@ajitbohra
Copy link
Member

Thanks for reporting this we will optimize this.

@nguyentranchung
Copy link

@tornadofay

Currently I am using this package and maintaining it. Everyone can use it here
https://github.com/nguyentranchung/laravel-decomposer

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants