-
Notifications
You must be signed in to change notification settings - Fork 62
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
release 3.2.0 breaks esphome web_server component #30
Comments
Specifically I think the mutex/bool for queue processing is a problem for the events server. So far ESP32 doesn't seem to be having issues, or nowhere near as significant as ESP8266. I can help debug, but not really sure what the best way to do that is. Add debug logging around the mutex/bool? Issue may be related to the following statement from the ESPHome documentation. "Additionally, each time a client connects to the event source the server sends out all current states so that the client can catch up with reality." That would be corroborated by anecdotal evidence that devices with more entities have worse issues with this. |
Added a print statement and can see that when the web_server page fails to load, it is stuck in an infinite loop at this line:
As soon as I close the browser page trying to load events, the loop stops. When the web page loads properly, that return never executes. |
When the web interface fails to load properly, the message queue fills up to 32 and then stays there forever. Never goes down at all. |
It appears as though, when the page fails to load, the _messageQueue_processing boolean is already set to true by default without the _runQueue function ever executing. |
This PR was merged so issue should be fixed in 3.2.1. Still needs to be added into ESPHome though. |
please see this issue esphome/issues#5793
The text was updated successfully, but these errors were encountered: