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

youtube crash after claiming the token grant #1326

Closed
btlechowski opened this issue Sep 27, 2018 · 12 comments · Fixed by brave/brave-core#527
Closed

youtube crash after claiming the token grant #1326

btlechowski opened this issue Sep 27, 2018 · 12 comments · Fixed by brave/brave-core#527

Comments

@btlechowski
Copy link

btlechowski commented Sep 27, 2018

IMPORTANT: The current workaround for this issue is to disable Brave Rewards

Steps to Reproduce

  1. Clean profile
  2. Enable rewards
  3. Claim the grant
  4. Browser youtube(search/watch videos)

Note: The issue is random but easily reproducible

1326_fail

For reference here are the steps used in the recording:

  1. Clean profile
  2. Enable rewards
  3. Claim the grant
  4. Open youtube.com in new tab
  5. Search for bart in youtube search
  6. Click on bart baker

Actual result:

Crash
Crash link: https://stats.brave.com/dashboard#crash/5bad34ae101f57001ebaca68
Crash report ID: 2f99acab89ff4b73
Local Crash ID: af5c42c4-36db-49fa-93f1-08ccd8bc3287
Crash dmp: Win10_5bad34ae101f57001ebaca68.zip
Win 10 Crash provided by @srirambv .

Win7_a215e857-ca96-420e-9531-f34c5cc94849.zip
Blocked on uploading crashes on Win7: #1188

Expected result:

No crash

Reproduces how often:

easily reproducible, but needs to browser youtube a bit.

Brave version (chrome://version info)

Brave 0.55.10 Chromium: 70.0.3538.22 (Official Build) beta (64-bit)
Revision ac9418ba9c3bd7f6baaffa0b055dfe147e0f8364-refs/branch-heads/3538@{#468}
OS Windows 7

Reproducible on current release:

Brave 0.55.10 Chromium: 70.0.3538.22 (Official Build) beta (64-bit)
Revision ac9418ba9c3bd7f6baaffa0b055dfe147e0f8364-refs/branch-heads/3538@{#468}
OS Windows 7

Reproduced by @srirambv

cc @brave/legacy_qa @bbondy

@NejcZdovc
Copy link
Contributor

@btlechowski could you please add exact steps, it's not crashing for me with those steps

@NejcZdovc
Copy link
Contributor

also is this limited to only windows?

@NejcZdovc
Copy link
Contributor

is this limited to youtube or the same things happens when you visit a regular site?

@srirambv
Copy link
Contributor

@Miyayes reported issues from community on #1327

+3 reproductions of this in the community (Windows).
Some users replicated 1326 quite identically.

@bbondy
Copy link
Member

bbondy commented Sep 28, 2018

This needs a local crash ID that was uploaded please.

@btlechowski
Copy link
Author

btlechowski commented Sep 28, 2018

@NejcZdovc Unfortunately, there is no reliable STR. It is a random issue, but it is easily reproducible.

I have reported this issue on Windows 7, but @srirambv reproduced on Windows 10. Also community reports that it is happening on Windows.

@LaurenWags @kjozwiak Can you check Mac?
I have checked Brave v0.55.10 on Ubuntu 17.10 and it does not crash. @srirambv can you check on your linux machine?

@bbondy updated main issue with crash dmps

@btlechowski
Copy link
Author

Also randomly reproduced on twitch.tv
Crash dmp:
Win7_twitch_02d97cdb-456e-4039-b915-39453a40b726.zip

Steps to Reproduce

  1. Clean profile
  2. Enable rewards
  3. Claim the grant
  4. Go to twitch.tv
  5. Play stream on the main page.
    1326_fail2

@srirambv
Copy link
Contributor

@bbondy local crash id's for the crash
image

bridiver added a commit to brave-intl/bat-native-ledger that referenced this issue Sep 29, 2018
bridiver added a commit to brave/brave-core that referenced this issue Sep 29, 2018
…blisherInfo to SavePublisherInfoOnFileTaskRunner. Explicitly make a copy before binding the callback

fix brave/brave-browser#1326
@snowbound
Copy link

Sorry but in my case Rewards was never enabled yet every time I start Brave up on WIndows7 I get "brave api keys are missing. Some functionality of brave will be disabled" this was a clean install which simply imported my Chrome data in. I then added Chrome store extensions into the browser.

@kjozwiak
Copy link
Member

kjozwiak commented Oct 4, 2018

Sorry but in my case Rewards was never enabled yet every time I start Brave up on WIndows7 I get "brave api keys are missing. Some functionality of brave will be disabled" this was a clean install which simply imported my Chrome data in.

This will be resolved by #1152

I then added Chrome store extensions into the browser.

Can you reproduce this reliably with a specific extensions? If you have exact steps to reproduce, we can take a look and see if it's the same crash or something completely different.

@snowbound
Copy link

If memory serves me the extension adding did not influence the crash it was on that I did a clean install, imported my Chrome data and then added extensions. That is all I did before I saw the "brave api keys are missing. Some functionality of brave will be disabled"

@btlechowski
Copy link
Author

Verification passed on

Brave 0.55.11 Chromium: 70.0.3538.35 (Official Build) beta (64-bit)
Revision 28dcb499844fa40c28d5f62e337876cb936f79f5-refs/branch-heads/3538@{#678}
OS Windows 7

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