-
-
Notifications
You must be signed in to change notification settings - Fork 4k
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
Show admin warning when php-intl is missing #12767
Comments
Thank you for testing 👍 See #10655 for details and solution. |
@danielkesselberg However same question as in the other topic, perhaps instead of repeatedly within Nextcloud log (only), the error could be shown either:
Currently it leads to overspammed logs without users forced to recognize fast. E.g. web interface works well and I had missing intl for several days, at last since the time when I installed NC15 Beta 2 and didn't recognize until now. |
I see your point. The message is generated by https://github.com/tchwork/utf8/blob/master/src/Patchwork/Utf8/Bootup/intl.php a 3rdparty library used by nextcloud. There are two ways to get rid of this error message: A) write a patch and apply it for the library in https://github.com/nextcloud/3rdparty/ (not popular because you have to reapply the patch with each upgrade but sometimes there is no other way) B) hook into the error_logger and check if a error message with category E_USER_DEPCREATED match Hmm. I think we should add |
@danielkesselberg
Actually I though I read already somewhere that About error handling, perhaps lower priority for now:
NB: It is a pain to navigate through blown log files on e.g. RPi to even find, copy&paste the errors. |
Are u sure about that? I could not reproduce it with master. |
@danielkesselberg After installing Perhaps going off-topic now: When checking the logs, I now found that most properly one of my events failed to sync and the daily backup procedure (incl. maintenance mode) then broke it, lead to a corrupted calendar entry. I am trying to fix this now. So it MIGHT be possible, but from my point of view unlikely, that the failing Windows client is related with the initial CalDAV sync/database error.
Only relation I could think of:
Damn that I purged the logs after fixing |
As an additional vote to do something about this logs: https://help.nextcloud.com/t/nextcloud-deletes-my-files-on-server-every-few-hours/42438/9 On shared hostings users are not always able to install this module and hosting provider might not react fast, if ever. As long as the internal fallback is working, the repeated warning in logs should be suppressed/handled which otherwise will make it hard to find more urgent logs related to other issues as mentioned in the links topic. |
Fix is in #12821 |
My comment, why I think it's not the best solution 😉: #12821 (comment) |
My comment why I still think it is a good solution: #12821 (comment) Anyways: Thanks for this feedback - it's absolutely necessary to get also an outside view and a need to explain itself again to know that it's still the right direction where we are going. 😃👍 |
Steps to reproduce
Expected behaviour
No repeating error messages
Actual behaviour
You are using a fallback implementation of the intl extension. Installing the native one is highly recommended instead. at /var/www/nextcloud/3rdparty/patchwork/utf8/src/Patchwork/Utf8/Bootup/intl.php#18
appearing several times a second.
Obviously leads to Nextcloud Windows client failing to sync (just check, no actual sync). After installing
php-intl
the error spam stops and desktop client succeeds to sync.Not sure why I didn't have this module installed (is actually on my list..), so great to throw an error about this. But a single message or warning on admin panel is enough, not a massive repeating spam of this.
€: Just found now that this was already the case with Beta 2, recognized it just now after RC2 update, because Nextcloud Windows client failed. On Beta 2 it was appearing a bid irregularly 2-10 times a minute. After RC2 upgrade it appeared 2-3 times a second. I could imagine that it was triggered by client sync attempt, since it failed?
Server configuration
Operating system: Raspbian Buster
Web server: Apache 2.4
Database: MariaDB 10.1
PHP version: 7.2
Nextcloud version: 15 RC2
Updated from an older Nextcloud/ownCloud or fresh install: Updated from 15 Beta 2
Where did you install Nextcloud from: Official download source
List of activated apps:
App list
Nextcloud configuration:
Config report
Are you using external storage, if yes which one: no
Are you using encryption: no
Are you using an external user-backend, if yes which one: no
Client configuration
Browser: Opera 59
Operating system: Windows 10 1809
Logs
Nextcloud log (data/nextcloud.log)
Nextcloud log
The text was updated successfully, but these errors were encountered: