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

Improve Opcache detection #19191

Merged
merged 1 commit into from
Apr 15, 2020
Merged

Improve Opcache detection #19191

merged 1 commit into from
Apr 15, 2020

Conversation

laurisb
Copy link
Member

@laurisb laurisb commented Jan 29, 2020

Improve Opcache detection by checking if extension is loaded, as opcache_get_status() is commonly disabled on shared hosting servers

Improve Opcache detection by checking if extension is loaded, as opcache_get_status() is commonly disabled on shared hosting servers
@kesselb kesselb requested a review from rullzer January 29, 2020 10:13
@kesselb
Copy link
Contributor

kesselb commented Jan 29, 2020

Thanks 👍

@kesselb kesselb added this to the Nextcloud 19 milestone Jan 29, 2020
@kesselb kesselb added 3. to review Waiting for reviews bug enhancement and removed bug labels Jan 29, 2020
Copy link
Contributor

@kesselb kesselb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

php -r "var_dump(extension_loaded('Zend OPcache'));"
bool(true)

php -r "var_dump(function_exists('opcache_get_status'));"
bool(true)

php -d disable_functions=opcache_get_status -r "var_dump(extension_loaded('Zend OPcache'));"
bool(true)

php -d disable_functions=opcache_get_status -r "var_dump(function_exists('opcache_get_status'));"
bool(false)

This was referenced Apr 4, 2020
@ChristophWurst
Copy link
Member

Tested and works!

@ChristophWurst ChristophWurst added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Apr 15, 2020
@rullzer rullzer merged commit 19171d7 into nextcloud:master Apr 15, 2020
@welcome
Copy link

welcome bot commented Apr 15, 2020

Thanks for your first pull request and welcome to the community! Feel free to keep them coming! If you are looking for issues to tackle then have a look at this selection: https://github.com/nextcloud/server/issues?q=is%3Aopen+is%3Aissue+label%3A%22good+first+issue%22

@laurisb laurisb deleted the patch-1 branch April 15, 2020 09:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release Ready to be released and/or waiting for tests to finish enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants