-
-
Notifications
You must be signed in to change notification settings - Fork 108
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
Many exceptions since update to 0.18 #1281
Comments
I think this is similar to the discussion comment in I added additional logging which might help us narrow this down. |
Also can you let me know what notification channels you have setup? I'm assuming just Telegram and Mail? |
In 0.18.1 the same: I use: Mail, Telegram and DB. But mail & dB only on Thresholds but telegram for every test.
|
Addon Info: the telegram message was send 3 times for the same test, I think that’s a follow up error because of the exception… |
@AleksCee what are your thresholds set to? I've got the following and can't replicate the error:
Threshold's I have download, upload and ping all set to trigger. |
If I understand the code right in SendSpeedtestThresholdNotification.php You return an empty array if no threshold occurred so in the $failed array is a list of empty arrays if there was no threshold?
|
I have done a little test:
so if an array in array was count the result is false in the if check and then the name key is missed. |
So lines 57-61 should prevent an empty array from continuing the process. Basically a count of 0 = false which causes the process to end. What's odd is I've replicated your settings locally and on my Synology NAS running |
Got it, the nested arrays while empty are being counted which shouldn't happen. Working on the fix now... |
I think it’s not count 0 in this case if [] returned an push to the array see at mein Mini Test before. [[]] is not 0 php -r ' $failed = []; array_push($failed,[]); var_dump(count($failed));'
|
Sorry i was to slow with my test showing… only tipping on iPad right now an use a ssh terminal on iPad to test on the nas… but I think you got it now while I’m writing the laste explanation ;-) sorry for my speed with iPad tipping. |
I'm taking an evening off from code for beers. Should get this fixed this weekend though. |
Also found all threshold notification (mail, telegram, discord) not working from updated to 0.18.0 version ... |
@alexjustesen |
Not now, the exceptions comes in a special case. You can try to go around this, if you turn off threshold still the fix comes after the beer-timeout (sorry I’m getting funny) from @alexjustesen |
this is only a grep for exception, if you need more of the thread-dump, please let me know.
The text was updated successfully, but these errors were encountered: