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

content.js "could not establish connection" errors in the console #289

Closed
englehardt opened this issue Apr 24, 2019 · 4 comments
Closed
Labels

Comments

@englehardt
Copy link
Collaborator

See #288 for an example. JavaScript error: moz-extension://6cbcf7fe-c654-4558-bfa1-bcc2f8a7769c/content.js, line 983: Error: Could not establish connection. Receiving end does not exist.

Possible cause? https://stackoverflow.com/questions/44673632/firefox-webextension-error-could-not-establish-connection-receiving-end-does-n

@englehardt englehardt added the bug label Apr 24, 2019
@nhnt11
Copy link
Contributor

nhnt11 commented Apr 25, 2019

This happens when the JS instrument is not enabled - we are injecting the instrumentation anyway but nothing is listening for the messages in the privileged scope.

The solution is to either only inject our instrumentation if the JS instrument is enabled in the config, or to silently ignore this error by wrapping the block in a try/catch. The problem with the former is that it will likely cause an async delay while injecting our script which will cripple its utility. The problem with the latter is that it will suppress legitimate errors.

@englehardt
Copy link
Collaborator Author

We should figure out how to do the former. The content.js script mucks with a lot of the JS APIs, so there may be crawls where we want it disabled (especially if we're not collecting JS data for those crawls).

@tooflop101
Copy link

tooflop101 commented Apr 26, 2019

I solved the problem #288 by installing the repository 0.8 by downloading it from the link. Errors occurred when installing via the console of an unknown version through the command git clone.Version 0.8 is stable.

@motin
Copy link
Contributor

motin commented Jul 4, 2019

This was fixed in #375 along with #374

@motin motin closed this as completed Jul 4, 2019
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

4 participants