Skip to content
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

Need to verify if websocket data is revceived #65

Closed
forkineye opened this issue Aug 18, 2017 · 4 comments
Closed

Need to verify if websocket data is revceived #65

forkineye opened this issue Aug 18, 2017 · 4 comments
Labels

Comments

@forkineye
Copy link
Owner

forkineye commented Aug 18, 2017

Sometimes, configuration data fails to show up in the web UI. Root cause is unknown, however configuration receipt verification / re-querying should be done to prevent this from happening.
Example: https://diychristmas.org/vb1/showthread.php?7969-ESPixelStick-WiFi-Renard-amp-Pixel-Control-from-XLights&p=88776&viewfull=1#post88776

@forkineye forkineye added the Bug label Aug 18, 2017
@cinoan
Copy link
Contributor

cinoan commented Aug 18, 2017

Traced it to ESPAsyncWebServer before but didn't get a fix and haven't had chance to look recently - me-no-dev/ESPAsyncWebServer#138

Sending one message, e.g
ws.send('E1#G1#G2');
and handling that rather than the current
ws.send('E1'); // Get html elements ws.send('G1'); // Get Config ws.send('G2'); // Get Net Status
would probably avoid the issue.

@forkineye
Copy link
Owner Author

@cinoan Appreciate the help, thanks! I'll look into it.

@forkineye
Copy link
Owner Author

@cinoan I'm having issues re-creating this locally. It looks as if it may have fixed here - me-no-dev/ESPAsyncWebServer#144. They have since fixed the IE issue as well so my fork of ESPAsyncWebServer should no longer be required. Any chance you can see if the latest ESPAsyncWebServer upstream still has the issue?

@cinoan
Copy link
Contributor

cinoan commented Sep 7, 2017

@forkineye Sorry I hadn't got back to this before. Just tried updating the ESPAsync libraries and still getting the same issue with messages being missed. Short on time at the moment but will try to find some to look at it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants