-
Notifications
You must be signed in to change notification settings - Fork 150
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
entrypoint: enable configuring PHP memory limit #374
Conversation
Adds an environment variable, PHP_MEMORY_LIMIT, which overrides the default memory limit as shipped by PHP. Closes wallabag#124
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your contribution, could you update the README so people will know how to use it?
Updates the readme with documentation for the new PHP_MEMORY_LIMIT environment variable
@j0k3r Added documentation to the README; sorry I didn't notice it should be there! |
Hi, first off 👍 for this change (that actually would allow me to re-fetch some of my entries as they systematically blow the memory limit up). Naive question but, any update on merging this PR ? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not a big fan of sed-ing the php.ini
file as there are other ways to override php configuration settings.
For example we could add a file 99-wallabag.ini
in /etc/php81/conf.d/
/var/www/wallabag # echo memory_limit=256M > /etc/php81/conf.d/99-wallabag.ini
/var/www/wallabag # php -i | grep -i memory
Zend Memory Manager => enabled
memory_limit => 256M => 256M
Collecting memory statistics => No
Supersedes #374 Fixes #124 Signed-off-by: Kevin Decherf <[email protected]>
Supersedes #374 Fixes #124 Signed-off-by: Kevin Decherf <[email protected]>
Superseded by #403 |
Adds an environment variable,
PHP_MEMORY_LIMIT
, which overrides the default memory limit as shipped by PHP.This environment variable, if specified, should be in the format normally expected by PHP (such as 256M or 1G). If unspecified, it defaults to the memory limit specified in php.ini at time of writing, which is 128M. If desired, a more specific default value can be chosen.
Supercedes #125
Supercedes #166
Closes #124