-
Notifications
You must be signed in to change notification settings - Fork 310
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
Background updates eventually hang #2638
Comments
When you open the app, does the sensors get updated correctly? I mean, when it is in foreground all works? |
It does not update when the app is in the foreground. I do not have local push enabled, or an internal URL configured. |
Did you observe a pattern for when does it stop reporting? In background I know for sure the sensor updates wont be live without local push enabled but foreground should be working properly. |
Unfortunately there doesn't seem to be a pattern to it, I have no way to reproduce. Is there anything useful I can capture the next time it happens? |
Your logs. You can export going in Debugging > export |
Can you check the latest testflight beta? It may have fixed this |
I'll update to the latest beta and continue to monitor! 🤞🏻 |
POSIXErrorCode(rawValue: 28): No space left on device) Is thrown constantly by the app when the WS connection breaks was something changed in the test flight version related to URLsessions or any kind of networking introspection libraries? |
@bgoncal let me know if there is anything you need to assist in debugging this. |
This looks related, I'll investigate next week |
RC still has the issue. I’ll also note that when this happens and you open the app after it has been in the background it doesn’t connect to HA at all. It fails and HA logs the following:
hard closing the app and reopening it resolves the issue. I have been running test flight versions since they started getting published again and this just started with this cycle of work: after CarPlay release version so maybe 2 or 3 test flight pushes back. |
I just thought of something else. The timing could possibly coincide with the release of iOS 17.4 |
Can you consistently reproduce the issue in some way? Like rebooting HA with the App open for example. I would like to have the same issue happening on my device to investigate it better, for now I couldn't reproduce |
Having a Shortcut automation which fires on power connect and disconnect to "Update Sensors" in the background is the way this manifests most often for me. |
Just leaving the app alone in the background for like 30 mins is all it takes for me |
@dmulcahey Im checking your logs, when they happened were you somewhere http://192.168.1.78:8123 was accessible? (your home probably) or were you using external URL? |
I work from home and am generally here. I just opened the app from the background again. Last open was around the time of my last reply and it happened again. I’ll get new logs for you in a few. Can you give me somewhere to send an iCloud link so I can get you the full bundle from the app? |
Are you in discord? You can ping me there @bgoncal2 |
Im preparing a testflight build with a possible fix , when you see 2024.612 in testflight you can test it |
Just saw this update and installed it. Will report back |
Initial testing looks good. Let’s see how it does overnight |
Everything is still working. What was the issue? |
I'm still investigating but initially I believe the issue is that StarScream lib we use for websocket interaction creates one URLSession per request, which in a retry loop goes over iOS limits. I believe this wasn't happening in some updates ago because we were still supporting iOS 12 at that time and this limit was not applied. |
Heads up that this fix is not on our main branch yet, so if you see new builds coming in testflight, they may revert this fix |
@lwis can you also check if this works for you? |
Updated this morning, so maybe too early to tell, but I've not run into any issues yet. |
Feel free to reopen this issue if needed, I'm closing this issue for now and planning the next release to include the fix |
This is still happening, please let me know if there's any way I can help! |
Im doing some clean up around shortcuts, widgets and intents in general, I'll let you know when you can test it again |
@bgoncal I have the same behaviour with the current beta 2024.696 and iOS 17.5.1. Has there been any progress in fixing the bug? |
Do you see anything in your logs? Like 503 error? |
Interesting. I've just done some research. If I connect the app directly locally via IP, the update works. So it has something to do with the external connection, which I set up via Cloudflare Tunnel. |
Do you have more than 1 server? All of them have external access enabled? Do you see webhook error 503 in your logs? |
I’m experiencing the same issue where the update shortcut only works locally. The HA app is able to update the sensors periodically while it is completely closed and does an immediate push after opening the HA app. I have 2 servers and only one server has external access enabled via Cloudflare Tunnel. I don’t see 503 in my logs but I do see POSIX Error 57. I’ll try to upload my logs sometime soon if that will help. |
Some more information about my setup:
Here are my logs |
iOS device model, version and app version
Model Name: iPhone 15 Pro
Software Version: 17.4
App version: 2024.2
Home Assistant Core Version
2024.3.0
Describe the bug
Eventually background sensor updates stop happening, running the "Update sensors" shortcut times out. Force closing the app brings everything back to life and the shortcut starts working again.
To Reproduce
Have iOS app pushing sensor updates in background for a ~period of time.
Expected behavior
App continues to function without repeatedly force closing.
The text was updated successfully, but these errors were encountered: