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

"DevTools failed to parse SourceMap" warning #8281

Closed
Nuc1eoN opened this issue Feb 17, 2020 · 16 comments · Fixed by brave/ethereum-remote-client#194
Closed

"DevTools failed to parse SourceMap" warning #8281

Nuc1eoN opened this issue Feb 17, 2020 · 16 comments · Fixed by brave/ethereum-remote-client#194

Comments

@Nuc1eoN
Copy link

Nuc1eoN commented Feb 17, 2020

In Brave v1.3.115 if you open any page, go to the Inspector and open the console, there are the following warnings:

DevTools failed to parse SourceMap: chrome-extension://odbfpeeihdkbihmopkbjmoonfanlbfcl/sourcemaps/contentscript.js.map
DevTools failed to parse SourceMap: chrome-extension://odbfpeeihdkbihmopkbjmoonfanlbfcl/sourcemaps/inpage.js.map

At first I thought it's a website issue but it really seems to be tied to the brave browser.

@minaiism
Copy link

Same issue here :(

@EugenioAntonioBonanno
Copy link

EugenioAntonioBonanno commented Feb 27, 2020

I am experiencing the same issue. I thought it might be an extension problem but I've checked all my extensions and there are no issues. I am running Brave (stable) for Linux on Ubuntu.

@bsclifton bsclifton added feature/dev-tools feature/web3/wallet Integrating Ethereum+ wallet support labels Feb 27, 2020
@bsclifton
Copy link
Member

bsclifton commented Feb 27, 2020

Looks like this is caused by the Crypt Wallets extension (odbfpeeihdkbihmopkbjmoonfanlbfcl)
cc: @bbondy @ryanml

@ryanml
Copy link
Contributor

ryanml commented Feb 27, 2020

@Nuc1eoN @minaiism @EugenioAntonioBonanno can you folks confirm whether or not you've navigated to brave://wallet at any time?

@EugenioAntonioBonanno
Copy link

EugenioAntonioBonanno commented Feb 28, 2020 via email

@bbondy bbondy added the priority/P4 Planned work. We expect to get to it "soon". label Feb 28, 2020
@vdavid
Copy link

vdavid commented Mar 5, 2020

I'm quite sure I haven't been at brave://wallet, and I definitely wasn't in the past 1 month. I ended up here because I'm also seeing the warnings in the console. I hope it helps.

@minaiism
Copy link

minaiism commented Mar 5, 2020

Same as @vdavid .I haven't been at brave://wallet whatsoever.

@bsclifton
Copy link
Member

@ryanml even if you don't enable Crypto Wallets, the extension is always running. Visible in the Task Manager (Hamburger => More Tools => Task Manager)
Screen Shot 2020-03-05 at 4 57 59 PM

@bbondy
Copy link
Member

bbondy commented Mar 6, 2020

Hey @bsclifton that's not right for a fresh profile that hasn't navigated to brave://wallet on Release channel up to Nightly.
Nightly, Beta, and Dev also has some MetaMask compatibility work which will require you to select Crypto Wallets as your Dapp provider before it will display any warnings.

@bsclifton
Copy link
Member

ah - ok my bad; I believe I did visit brave://wallet. I didn't realize it would run the extension even if Crypto Wallets is disabled / not used

@marcuswhybrow
Copy link

Disabling Crypto Wallets worked for me:

Brave

@zaxbux
Copy link

zaxbux commented Mar 27, 2020

There's a source map comment in the affect files, but the source maps themselves are missing. The developers should disable source map creation in their asset build process.

Edit: Removing those comments fixed this issue.

@cdock1029
Copy link

Upstream issue

Looks like more errors are exposed (if source-map files are missing, error will now show where it didn't before)

And, currently source-maps from extensions aren't loaded even if they are provided.

I'm seeing this error for 1PasswordX extension, having disabled crypto wallets already.

@mrose17
Copy link
Member

mrose17 commented Nov 12, 2020

@rebron - may I ask that someone remove the comments referencing a source map (or install the source map files)? the warning is rather annoying and distracts me when i'm trying to debug things in the browser... thanks!

@bbondy bbondy added priority/P3 The next thing for us to work on. It'll ride the trains. and removed priority/P4 Planned work. We expect to get to it "soon". labels Nov 12, 2020
ryanml added a commit to brave/ethereum-remote-client that referenced this issue Nov 13, 2020
@ryanml ryanml self-assigned this Nov 13, 2020
@mrose17
Copy link
Member

mrose17 commented Nov 13, 2020

Thank you!

@srirambv
Copy link
Contributor

Verification passed on

Brave 1.17.73 Chromium: 87.0.4280.67 (Official Build) (64-bit)
Revision 0e5d92df40086cf0050c00f87b11da1b14580930-refs/branch-heads/4280@{#1441}
OS Linux
Component 1.0.17 dev
  • Verified no error/warnings about SourceMap in devtools irrespective of whether Crypto Wallet was enabled or not

Verification passed on

Brave 1.19.16 Chromium: 87.0.4280.66 (Official Build) nightly (64-bit)
Revision fd98a29dd59b36f71e4741332c9ad5bda42094bf-refs/branch-heads/4280@{#1432}
OS Windows 10 OS Version 1809 (Build 17763.1577)
Component 1.0.17 dev
  • Verified no error/warnings about SourceMap in devtools irrespective of whether Crypto Wallet was enabled or not

Verification passed on

Brave 1.18.56 Chromium: 87.0.4280.60 (Official Build) dev (x86_64)
Revision 12697cfeb273d7de95cf9b18350d2c457f58224c-refs/branch-heads/4280@{#1352}
OS macOS Version 10.15.7 (Build 19H2)
Component 1.0.17 dev
  • Verified no error/warnings about SourceMap in devtools irrespective of whether Crypto Wallet was enabled or not

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment