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

Extension crashing non-Metamask pages #5621

Closed
jadbox opened this issue Oct 26, 2018 · 2 comments
Closed

Extension crashing non-Metamask pages #5621

jadbox opened this issue Oct 26, 2018 · 2 comments

Comments

@jadbox
Copy link

jadbox commented Oct 26, 2018

I have users reporting my website crashes, but if they disable their Metamask extension, it will load correctly. The website does not include or use anything related to Metamask (it's not a dapp and has nothing to do with crypto).

Here's a captured console log of when the crash happens:

Uncaught TypeError: chrome.extension.getBackgroundPage is not a function
    at content.min.js:1
inpage.js:1 ATTENTION: In an effort to improve user privacy, MetaMask will stop exposing user accounts to dapps by default beginning November 2nd, 2018. Dapps should call provider.enable() in order to view and use accounts. Please see https://bit.ly/2QQHXvF for complete information and up-to-date example code.
(anonymous) @ inpage.js:1
site-globals.css:1 Failed to load resource: the server responded with a status of 404 ()
options.min.js:1 Uncaught TypeError: Cannot read property 'getAll' of undefined
    at Object.fetchAllPermittedSites (options.min.js:1)
    at HTMLDocument.<anonymous> (options.min.js:1)

The website is in private beta currently, but if you give me a way to reach out, I can get you access. For me using Chrome on Windows and Mac with Metamask installed, I have yet to reproduce the problem.
I'll try to gather more information...

@jadbox
Copy link
Author

jadbox commented Oct 26, 2018

One user's system:

chrome
Windows PC
Google Chrome está actualizado
Versión 70.0.3538.77 (Build oficial) (64 bits)

@jadbox
Copy link
Author

jadbox commented Oct 26, 2018

I haven't been able to reproduce the issue, but it would be nice to have the MM warning log removed.

@jadbox jadbox closed this as completed Oct 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant