-
Notifications
You must be signed in to change notification settings - Fork 31
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
Intermittent failure of repl connection #17
Comments
Any kind of transient failure like this very often ends up being cruft having built up in various directories that the compiler / REPL use to store intermediate bits. Try blowing away |
I tried that. Also tried with a freshly pulled copy of the Austin repo. With Chrome, I've seen no successful attempts for the past day's focused testing, and very few successes before that. With Firefox, things are consistently working (though only a few attempts so far). I will continue testing later today (guess I'll start by clearing Chrome's state). So far, all I can say for sure is that I'm seeing a different pattern of network traffic when I compare Firefox with the failing Chrome cases. Can you reliably run this in Chrome? Actually, to be precise, I'm running Chromium: "Version 28.0.1500.71 Ubuntu 12.10 (28.0.1500.71-0ubuntu1.12.10.1)" |
Chrome is actually my default catch-all test browser (including hosting most of my ClojureScripting). I don't think I've ever run chromium. |
|
Got it! The issue was not chrome vs chromium; that would have been too bizarre. But, at least that set me in the right direction. The real issue was that I was logged in to my Google account, which dragged along all my chrome/chromium extensions. Once I realized that, it was easy to isolate the culprit: the Google Voice extension. A frustrating SOB of a bug; who-woulda-thought to look there. But, at least I'm good now. Do you want me to submit a patch with a warning line for the README.md? |
That is really strange. A warning is worthwhile, but I'd like to know what about the extension was interfering; presumably whatever it's doing isn't unique among chrome extensions…? |
I agree 100%. I can do the first steps of describing the visible breakage, but I don't have the tools to poke deeper; I'm less than a rank novice at Chrome's architecture and know zero about Google Voice internals. Do you know anyone who might have a good handle on attacking this kind of problem? Of course, it would also be extremely useful for someone else to verify this bug, and make sure that it's not an interaction of GV with something else in my environment. Do you have the time to do so? |
Some more info which may mean more to you than to me: In both cases (Chromium with Google Voice enabled or disabled, running a clean browser-connected-repl-sample), the first four packets are the same:
With response
Then, the working case (GV disabled) immediately sends a fifth packet:
This, of course, sits in the long-poll with no response until I evaluate something in the REPL. The failing case never sends this packet at all. |
This (goofy) extension breaks in the same way: https://chrome.google.com/webstore/detail/dictionary-of-numbers/ahhgdmkmcgahbkcbmlkpmmamemlkajaf |
With a second confirming case, I'm happy to have the warning in the patch on gh-18 shown in the main README as well, in the " Using other browser runtimes" section. |
Hoping that this is related to the use of CrossPageChannel, and therefore resolved by #50. |
Most of the time, I can't get browser-connected-repl-sample to work. But, I have seen it work occasionally, so I know my config is not totally wedged. Unfortunately, I haven't been able to get it to work at all recently, so I can't compare working vs broken cases to see what's going wrong.
The problem seems to be that the long-poll gets answered and the next one is not posted.
Until just now I've been using only Chrome, and the problem occurred most of the time. I just installed and tried with Firefox and things work on the first attempt. So, the problem may be chrome specific. I'll continue to debug, but just wanted to check in to see if this rings any bells.
The text was updated successfully, but these errors were encountered: