-
Notifications
You must be signed in to change notification settings - Fork 65
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
Websocket connection failed #75
Comments
It's the same here. I tested it on two different Windows 7 64 machines and on OSX. It makes debugging using the javascript console useless because it gets floatet with those error messages. Also it seems to slow down Theseus. |
Yeah apart from being annoying it throwing errors all the time can't be good for performance. If this is intended behaviour it should silently fail |
Patches welcome! I don't know how to fix this easily. The smallest-change fix (which I can't figure out) would be to squelch the error somehow, because it's completely benign. However, it happens asynchronously, so there's nothing to try-catch. I search the web every so often for a way and haven't found a fix. There's an error handler on the socket, but Chrome logs it anyway. A patch-it-over but still small fix would be to stop trying to connect when Theseus is disabled. At least then you could disable Theseus to mute these errors. This would be less surprising behavior for the automatic retries anyway. Eventually I hope to flip the connection so that the debugging server is run in Brackets and the debugged process repeatedly tries to connect to Brackets, instead of the other way around, which is how it works now. Then there'd be no connection failed errors in Brackets ever. :) |
Apperently those errors are by design https://groups.google.com/a/chromium.org/d/msg/chromium-html5/4G_XqPQQFX8/Yt4YN5DiQ0EJ post is from 2011 though. I agree that turning it around would be better, any ETA on that? 😜 |
ETA? I am still seeing this issue |
AdriVanHoudt + vincentfaller: If you could stop the warnings by temporarily disabling Theseus from the File menu, would that be enough to fix your workflow, or do you need the bigger fix? The small fix is to just check (Sorry for the delay. I was away for the summer.) |
@alltom tbh I use node-inspector now but temp disabling it when you turn of theseus in the menu would indeed be a fine solution for me |
I am experiencing this same issue. Agent-node.js:61 WebSocket connection to 'ws://localhost:8888/' failed: Error in connection establishment: net::ERR_CONNECTION_REFUSED repeatedly shows up in my console when theseus is installed, even when no program is running or when theseus is set to off. Is this going to be fixed soon? |
Fixed for me after reboot. |
While doing nothing Theseus keeps throwing errors in console
![](https://camo.githubusercontent.com/945e17fa4a13300bcfcfe7cc6fa78d3487d383cb411a8e6351de50007e240c72/687474703a2f2f7075752e73682f65556954642f353533666435323562302e706e67)
And in text. (1076 is the counter btw)
.
The text was updated successfully, but these errors were encountered: