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

inline tipping doesn't work under Twitter #7324

Closed
kjozwiak opened this issue Dec 11, 2019 · 6 comments · Fixed by brave/brave-core#4215
Closed

inline tipping doesn't work under Twitter #7324

kjozwiak opened this issue Dec 11, 2019 · 6 comments · Fixed by brave/brave-core#4215

Comments

@kjozwiak
Copy link
Member

kjozwiak commented Dec 11, 2019

Description

Inline tipping has stopped working under Twitter. When pressing the inline BAT button, you'll get the following errors in the browser console:

brave_rewards_panel_content_twitter.bundle.js:1 Failed to fetch tweet details for 1204589420617371649: Unexpected end of JSON input
(anonymous) @ brave_rewards_panel_content_twitter.bundle.js:1
Promise.catch (async)
l @ brave_rewards_panel_content_twitter.bundle.js:1
s.onclick @ brave_rewards_panel_content_twitter.bundle.js:1

brave_rewards_panel_content_twitter.bundle.js:1 Failed to fetch tweet metadata for [object HTMLDivElement]: SyntaxError: Unexpected end of JSON input
    at a (brave_rewards_panel_content_twitter.bundle.js:1)
(anonymous) @ brave_rewards_panel_content_twitter.bundle.js:1
Promise.catch (async)
s.onclick @ brave_rewards_panel_content_twitter.bundle.js:1

brave_rewards_panel_content_twitter.bundle.js:1 Cross-Origin Read Blocking (CORB) blocked cross-origin response https://api.twitter.com/1.1/statuses/show.json?id=1204589420617371649 with MIME type application/json. See https://www.chromestatus.com/feature/5629709824032768 for more details.
a @ brave_rewards_panel_content_twitter.bundle.js:1
async function (async)
a @ brave_rewards_panel_content_twitter.bundle.js:1
(anonymous) @ brave_rewards_panel_content_twitter.bundle.js:1
l @ brave_rewards_panel_content_twitter.bundle.js:1
s.onclick @ brave_rewards_panel_content_twitter.bundle.js:1

You'll get the following errors in the terminal:

[79025:775:1210/223655.310171:INFO:CONSOLE(1)] "Failed to fetch tweet details for 1204604280180789248: Unexpected end of JSON input", source: chrome-extension://jidkidbbcafjabdphckchenhfomhnfma/out/brave_rewards_panel_content_twitter.bundle.js (1)

Seems like something was introduced in C79 that's causing the issue. Errors in the browser console are mentioning a CORS issue.

Steps to Reproduce

  1. install & launch 1.1.20 CR: 79.0.3945.74
  2. enable Rewards and login into https://twitter.com/
  3. attempt to tip users/tweets via inline tipping

Actual result:

Clicking on the BAT icon via inline won't do anything and will produce the above errors mentioned above via the browser console and the terminal.

Expected result:

Clicking on the BAT icon via inline on Twitter should display the tipping banner.

Reproduces how often:

100% reproducible when using the above STR.

Brave version (brave://version info)

Brave 1.1.20 Chromium: 79.0.3945.74 (Official Build) (64-bit)
Revision 99b6d241cf84f7fa7434ae8a59f9e74eb1464280-refs/branch-heads/3945@{#888}
OS macOS Version 10.15.1 (Build 19B88)

Version/Channel Information:

  • Can you reproduce this issue with the current release? Reproducible
  • Can you reproduce this issue with the beta channel? Reproducible
  • Can you reproduce this issue with the dev channel? Reproducible
  • Can you reproduce this issue with the nightly channel? Reproducible

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields? No, still reproducible
  • Does the issue resolve itself when disabling Brave Rewards? N/A
  • Is the issue reproducible on the latest version of Chrome? N/A

Miscellaneous Information:

CCing @rebron @bsclifton @brave/legacy_qa @NejcZdovc

@kjozwiak kjozwiak added bug feature/rewards priority/P2 A bad problem. We might uplift this to the next planned release. QA/Yes QA/Test-Plan-Specified regression labels Dec 11, 2019
@kjozwiak kjozwiak added this to the 1.1.x - Release Hotfix 1 milestone Dec 11, 2019
@kjozwiak kjozwiak added this to New Issues in Rewards via automation Dec 11, 2019
@NejcZdovc NejcZdovc self-assigned this Dec 11, 2019
NejcZdovc added a commit to brave/brave-core that referenced this issue Dec 11, 2019
@NejcZdovc NejcZdovc moved this from New Issues to In Progress in Rewards Dec 11, 2019
@LaurenWags
Copy link
Member

inline tipping for both Reddit and GH are working as expected - when clicking on the inline Tip button the tip banner displays.

Brave 1.1.20 Chromium: 79.0.3945.74 (Official Build) (64-bit)
Revision 99b6d241cf84f7fa7434ae8a59f9e74eb1464280-refs/branch-heads/3945@{#888}
OS macOS Version 10.13.6 (Build 17G5019)

@btlechowski
Copy link

Reproduced on

Brave 1.1.20 Chromium: 79.0.3945.74 (Official Build) (64-bit)
Revision 99b6d241cf84f7fa7434ae8a59f9e74eb1464280-refs/branch-heads/3945@{#888}
OS Ubuntu 18.04 LTS

@kjozwiak
Copy link
Member Author

After speaking with @rebron and @bbondy, we'll move this out of it's not completed by the time #7314 is done. We'll want to get the CR out ASAP whenever it's available.

NejcZdovc added a commit to brave/brave-core that referenced this issue Dec 11, 2019
NejcZdovc added a commit to brave/brave-core that referenced this issue Dec 12, 2019
bsclifton pushed a commit to brave/brave-core that referenced this issue Dec 12, 2019
@LesterCovax
Copy link

Not sure if it's related, but I noticed that Brave (on Android at least) isn't targeting the correct Twitter user when opening the rewards panel while on someone's profile. For instance, if I go to my Twitter profile, and click the rewards button, it shows the information for a different user's profile that I had accessed the other day (even though I'm verified and all). Let me know if it's not related and I'll create a new issue if one doesn't exist.

Screenshot_20191212-150130

Also noticed (and I'm sure you get this a lot) that @brave itself isn't verified (on GH at least).

Screenshot_20191212-145407

@bsclifton
Copy link
Member

Fixed with brave/brave-core#4215

cc: @SergeyZhukovsky @samartnik @AlexeyBarabash RE: the above Android issue

@btlechowski
Copy link

btlechowski commented Dec 13, 2019

Verification passed on

Brave 1.1.21 Chromium: 79.0.3945.79 (Official Build) (64-bit)
Revision 29f75ce3f42b007bd80361b0dfcfee3a13ff90b8-refs/branch-heads/3945@{#916}
OS Windows 7 Service Pack 1 (Build 7601.24530)

Verified test plan from the description and brave/brave-core#4215.
Verified that I was able to tip the publisher.
Verified that the tip was visible under Tips on brave://rewards

Verification passed on

Brave 1.1.21 Chromium: 79.0.3945.79 (Official Build) (64-bit)
Revision 29f75ce3f42b007bd80361b0dfcfee3a13ff90b8-refs/branch-heads/3945@{#916}
OS Ubuntu 18.04 LTS

Verified test plan from the description and brave/brave-core#4215.
Verified that I was able to tip the publisher.
Verified that the tip was visible under Tips on brave://rewards

Verified passed with

Brave 1.1.21 Chromium: 79.0.3945.79 (Official Build) (64-bit)
Revision 29f75ce3f42b007bd80361b0dfcfee3a13ff90b8-refs/branch-heads/3945@{#916}
OS macOS Version 10.13.6 (Build 17G5019)

Screen Shot 2019-12-13 at 9 22 53 AM

* Verified normal banner displayed as well:

Screen Shot 2019-12-13 at 9 23 25 AM

  • Verified able to tip Twitter publishers from panel and tips displayed in Tips panel on brave://rewards

Screen Shot 2019-12-13 at 9 24 03 AM

Screen Shot 2019-12-13 at 9 39 00 AM

Screen Shot 2019-12-13 at 9 42 59 AM

Screen Shot 2019-12-13 at 9 45 48 AM

Verification PASSED on macOS 10.15.1 x64 using the following build:

Brave 1.1.21 Chromium: 79.0.3945.79 (Official Build) (64-bit)
Revision 29f75ce3f42b007bd80361b0dfcfee3a13ff90b8-refs/branch-heads/3945@{#916}
OS macOS Version 10.15.1 (Build 19B88)

Note: Used Production instead of Staging

  • Verified test plan from the description and Fixes inline tip for twitter brave-core#4215.
  • Verified that I was able to tip the publisher.
  • Verified that the tip was visible under Tips on brave://rewards
  • also checked GH & Reddit re: inline tipping

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

Successfully merging a pull request may close this issue.

6 participants