Setup warnings on fresh install

freenas / iocage-plugin-nextcloud

Artifact file(s) for nextcloud iocage plugin

18 stars 34 forks source link

Setup warnings on fresh install #7

Closed ghost closed 5 years ago

ghost commented 5 years ago

Hi. I recently set up Nextcloud and noticed it gives a number of warnings and recommendations in Settings>Administration>Overview. PHP environment variables seem to be the big one. Is this something that should be fixed on the plugin installation? I can go into the jail and manually tune these things but maybe they should be auto-configured on plugin installation

louizo commented 5 years ago

Agree, I think it should be fix during installation. I'm currently working on this (just started)... I will request merge for fix on php.ini first, if it's accepted will do other patches for more complex fix (such as imagick) and big int fixes

ghost commented 5 years ago

Thanks! I appreciate the quick response. I'll keep an eye out for your changes so I can make matching updates to my existing installation. I'm concerned about using the built-in updater tool while the environment variables are misconfigured. I don't know if there's any risk but I'd rather not chance it

miwi-fbsd commented 5 years ago

Thanks for pointing it out, Patch looked good.

ghost commented 5 years ago

@miwi-fbsd There are other issues in the first post aside from php opcache settings.. reopen or submit new issue?

Fix tested and working on my end for opcache. Thanks @louizo!

miwi-fbsd commented 5 years ago

@arcooke Please open a new pr.