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

PostProcessors timeout #1070

Closed
proArtex opened this issue Mar 2, 2018 · 1 comment
Closed

PostProcessors timeout #1070

proArtex opened this issue Mar 2, 2018 · 1 comment

Comments

@proArtex
Copy link

proArtex commented Mar 2, 2018

Q A
Bug Report? no
Feature Request? no
BC Break Report? no
RFC? yes
Imagine Bundle Version 2.0

OptiPngPostProcessor creates and runs new Process($processArguments) in processWithConfiguration() method, but it doesn't contain 4th argument $timeout in constructor, so it becomes 60s by default. This postprocessor may take over 60s, so it would be great to have ability to configure timeout of processors in some way.

@robfrawley
Copy link
Collaborator

robfrawley commented Mar 2, 2018

This is already in the works in #993. We'll see if we can't get it in the 2.0.0 release. If you want to discuss any specifics about implementations, please continue the discussion there: #993.

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

No branches or pull requests

2 participants