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

Rewards panel displaying incorrect publishers or none in certain cases when using Twitter #8289

Closed
kjozwiak opened this issue Feb 18, 2020 · 9 comments · Fixed by brave/brave-core#5113

Comments

@kjozwiak
Copy link
Member

Description

We've received several reports from the community mentioning that the Rewards panel sometimes displays and incorrect publisher in certain cases. In one of the examples, the user was seeing https://twitter.com/BrendanEich in the panel when viewing their DMs on Twitter.

There's also some cases where there's nothing appearing in the panel when viewing the panel. I've personally ran into this and have a Nightly that's experiencing this particular issue.

Possible related #5580 which was an issue in the past but was never really addressed as QA and others couldn't reproduce. Assumed it was fixed by proxy by another issue.

Steps to Reproduce

We'll need some information from the user experiencing this issue. I've tried reproducing a few times using 1.3.116 CR: 80.0.3987.106 without any luck. However, I did run into a case where my Nightly isn't displaying any publishers in the panel. I'll add STR once we have more info.

Actual result:

Screen Shot 2020-02-17 at 7 12 05 PM

image (1)

Screen Shot 2020-02-17 at 7 19 07 PM

Expected result:

Screen Shot 2020-02-17 at 7 10 04 PM

Screen Shot 2020-02-17 at 7 18 34 PM

Reproduces how often:

We'll need some more info/investigation to find STR so we can gauge how often this happens.

Brave version (brave://version info)

Brave 1.3.115 Chromium: 80.0.3987.87 (Official Build) (64-bit)
Revision 449cb163497b70dbf98d389f54e38e85d4c59b43-refs/branch-heads/3987@{#801}
OS macOS Version 10.15.3 (Build 19D76)
Brave 1.6.36 Chromium: 80.0.3987.106 (Official Build) nightly (64-bit)
Revision f68069574609230cf9b635cd784cfb1bf81bb53a-refs/branch-heads/3987@{#882}
OS macOS Version 10.15.3 (Build 19D76)

Version/Channel Information:

  • Can you reproduce this issue with the current release? Yes
  • Can you reproduce this issue with the beta channel? not 100% sure
  • Can you reproduce this issue with the dev channel? not 100% sure
  • Can you reproduce this issue with the nightly channel? Yes

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields? N/A
  • 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 @NejcZdovc @rebron @jonathansampson @yachtcaptain23 @BrendanEich @brave/legacy_qa

@kjozwiak
Copy link
Member Author

Another example of publishers not being displayed/appearing in the Rewards panel.

emptyPanelTwitter

@StephenTurley
Copy link

Hi, I'm the user in the first screenshot. Here is my info:

Brave 1.3.115 Chromium: 80.0.3987.87 (Official Build) (64-bit)
Revision 449cb163497b70dbf98d389f54e38e85d4c59b43-refs/branch-heads/3987@{#801}
OS Linux
JavaScript V8 8.0.426.16
Flash (Disabled)
User Agent Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/80.0.3987.87 Safari/537.36
Command Line /opt/brave.com/brave/brave --enable-dom-distiller --disable-domain-reliability --no-pings --extension-content-verification=enforce_strict --extensions-install-verification=enforce --sync-url=https://no-thanks.invalid --disable-sync --enable-features=PasswordImport,WebUIDarkMode,SimplifyHttpsIndicator --disable-features=AutofillServerCommunication,LookalikeUrlNavigationSuggestionsUI,AllowPopupsDuringPageUnload,NotificationTriggers,WebXR,AudioServiceOutOfProcess,SmsReceiver,VideoPlaybackQuality,WebXrGamepadModule --flag-switches-begin --flag-switches-end --disable-webrtc-apm-in-audio-service

@StephenTurley
Copy link

I'm unable to reproduce this on my other computer for some reason. Looks like the same version of Brave. Bug is present on PopOs/Gnome3 but not Manjaro/XFCE

Brave 1.3.115 Chromium: 80.0.3987.87 (Official Build) unknown (64-bit)
Revision 449cb163497b70dbf98d389f54e38e85d4c59b43-refs/branch-heads/3987@{#801}
OS Linux
JavaScript V8 8.0.426.16
Flash (Disabled)
User Agent Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/80.0.3987.87 Safari/537.36
Command Line /usr/lib/brave/brave --enable-dom-distiller --disable-domain-reliability --no-pings --extension-content-verification=enforce_strict --extensions-install-verification=enforce --sync-url=https://no-thanks.invalid --disable-sync --enable-features=PasswordImport,WebUIDarkMode,SimplifyHttpsIndicator --disable-features=AutofillServerCommunication,NotificationTriggers,AllowPopupsDuringPageUnload,WebXR,LookalikeUrlNavigationSuggestionsUI,SmsReceiver,AudioServiceOutOfProcess,VideoPlaybackQuality,WebXrGamepadModule --flag-switches-begin --flag-switches-end --disable-webrtc-apm-in-audio-service

@bsclifton bsclifton added the priority/P2 A bad problem. We might uplift this to the next planned release. label Mar 15, 2020
@NejcZdovc NejcZdovc assigned NejcZdovc and unassigned alexwykoff Mar 16, 2020
@bsclifton
Copy link
Member

Seems to be working after brave/brave-core#4948 was merged - will retest once we have a beta

@NejcZdovc NejcZdovc assigned emerick and unassigned NejcZdovc Mar 16, 2020
@kjozwiak
Copy link
Member Author

Seems to be working after brave/brave-core#4948 was merged - will retest once we have a beta

Seems like this is still happening. I can reproduce on my local profile using 1.8.12 CR: 80.0.3987.132 which includes brave/brave-core#4948. When we had brave/brave-core#4948 present in 1.7.x and 1.8.x, the reward panel wasn't working on most publishers which is working now 👍 However, Twitter still has the same problem. @bsclifton you were probably seeing the rewards panel being broken because of brave/brave-core#4948.

@jonathansampson
Copy link
Contributor

I just checked in Nightly (1.8.22) and was able to get bad results. I started by going to twitter.com/jonathansampson, and then clicked to visit another profile from there. The rewards state for "jonathansampson" was displayed on the next profile:

image

@LaurenWags
Copy link
Member

@emerick there really aren't good STR in the issue description, seems like solid ones were never found. Could you offer any guidance on a test plan? If not, is a general regression test of Twitter tipping sufficient? cc @kjozwiak @rebron

@emerick
Copy link
Contributor

emerick commented May 1, 2020

@LaurenWags Yeah this was a tough one to test and it seemed to be an intermittent issue at best. I think a general regression test of Twitter is definitely sufficient, perhaps focusing on multiple tabs.

@btlechowski
Copy link

btlechowski commented May 14, 2020

Verification passed on

Brave 1.9.63 Chromium: 81.0.4044.138 (Official Build) dev (64-bit)
Revision 8c6c7ba89cc9453625af54f11fd83179e23450fa-refs/branch-heads/4044@{#999}
OS Ubuntu 18.04 LTS

Verified clicking around twitter users shows proper user name in the Rewards Panel. Started with twitter.com/jonathansampson and https://twitter.com/BrendanEich
Verified different twitter users in different tabs
Verified different twitter users in different windows.

Noticed that it takes some time to load rewards verified status for a twitter user, especially after restart


Verification passed on

Brave | 1.9.66 Chromium: 81.0.4044.138 (Official Build) dev (64-bit)
-- | --
Revision | 8c6c7ba89cc9453625af54f11fd83179e23450fa-refs/branch-heads/4044@{#999}
OS | Windows 10 OS Version 1803 (Build 17134.1006)
  • Verified the description from the issue and confirmed that twitter publisher name is displayed correctly in BR panel as well as in tipping banner for verified publishers
  • Ensured that the verified checkmark is shown in BR panel and banner for the verified publisher
  • Ensured that twitter publisher name is displayed correctly in BR panel as well as in tipping banner for unverified publishers

image
image
image
image
image
image
image
image
image
image
image
image


Verified passed with

Brave | 1.9.70 Chromium: 81.0.4044.138 (Official Build) (64-bit)
-- | --
Revision | 8c6c7ba89cc9453625af54f11fd83179e23450fa-refs/branch-heads/4044@{#999}
OS | macOS Version 10.14.6 (Build 18G3020)
  • Verified twitter pages (ex. https://twitter.com/bravesampson) showed publisher both when I was logged into twitter and when I was not logged into twitter.
  • Verified opening multiple tabs w/ twitter users and switching between the tabs showed the expected publisher info in the rewards panel. Confirmed navigating to a different twitter user in the same page as another twitter user showed updated publisher info on the panel.
  • Verified able to tip verified twitter users and they displayed correctly in the tips panel on brave://rewards
  • Confirmed twitter users that I tipped were listed correctly in the database (confirmed twitter ID matches handle using http://gettwitterid.com/):

Screen Shot 2020-05-18 at 1 16 59 PM

Screen Shot 2020-05-18 at 1 17 27 PM

* Confirmed custom banner displayed for twitter publisher when applicable:

Screen Shot 2020-05-18 at 1 19 42 PM

* Tried same steps demonstrated in https://github.com//issues/8289#issuecomment-587214446 and confirmed expected publisher info shown each time.

twitterpubs

@rebron rebron changed the title rewards panel displaying incorrect publishers or none in certain cases when using Twitter Rewards panel displaying incorrect publishers or none in certain cases when using Twitter May 20, 2020
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.

10 participants