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

Brave Rewards displaying wrong Reddit usernames #15687

Closed
meejle opened this issue May 7, 2021 · 15 comments · Fixed by brave/brave-site-specific-scripts#59
Closed

Brave Rewards displaying wrong Reddit usernames #15687

meejle opened this issue May 7, 2021 · 15 comments · Fixed by brave/brave-site-specific-scripts#59
Assignees

Comments

@meejle
Copy link

meejle commented May 7, 2021

Description

I signed up to Brave Rewards Creator and claimed my Reddit account (among other social media accounts and websites that I own). I also submitted custom banners for each site/social media profile.

When trying to test this on Reddit (e.g., to see if the verified "tick" would appear on my profile page, whether my custom tip message would appear, etc.), I noticed various issues with usernames.

  1. Somehow, my Brave browser has got itself hung up on one particular Redditor, u/thinsoldier. It doesn't matter whose profile page I visit, if I click the Brave Rewards icon (in the address bar), it shows "thinsoldier on Reddit". Here's a screenshot of it happening on u/Brave_Support's profile page (https://www.reddit.com/user/Brave_Support/):

Screenshot 2021-05-07 at 18 21 59

Clicking "Send a Tip..." also shows u/thinsoldier as the recipient of the tip:

Screenshot 2021-05-07 at 18 25 21

But wait, it gets even weirder! Clicking the inline "Tip" buttons on Reddit seems to confuse things even more. Trying to tip on one of u/Brave_Support's comments, shows "thinsoldier on Reddit", but then correctly displays the username underneath as "u/Brave_Support". Also notice how it then shows a snippet of "thinsoldier's post" underneath:

Screenshot 2021-05-07 at 18 30 42

And weirder still: if I go to my own Reddit user page (I'm u/meejle -- remember that, it'll be important in a second) and click one of the inline "Tip" buttons... I get... brace yourself... "thinsoldier on Reddit"... and then underneath, "u/mikeleewrites"!

Screenshot 2021-05-07 at 18 34 38

Strangely enough, there is a u/Mikeleewrites (with a capital "M") on Reddit, but he isn't me. I only have one Reddit account, and it's always been u/meejle. HOWEVER: My Twitter username is @mikeleewrites (with a small "m"). I've claimed that Twitter account on my Brave Rewards Creator profile, so I can only imagine that's where Brave browser is somehow "finding" the "mikeleewrites" (with a small "m") username?

Yeah, sorry if you have to read this three or four times to make sense of it. I did my best.

I've tried clearing my history and cache, quitting and re-launching Brave, and rebooting (because why the heck not), but nothing has made a difference.

Strangely enough, using the old Reddit (e.g., https://old.reddit.com/user/meejle, https://old.reddit.com/user/Brave_Support) does seem to help the browser get over its obsession with u/thinsoldier (here it is correctly identifying u/Brave_Support):

Screenshot 2021-05-07 at 18 48 37

But it doesn't fix the other problem I'm having -- here it is correctly identifying the fact I'm a Brave Verified Creator, but wrongly referring to me as "mikeleewrites on Reddit" again:

Screenshot 2021-05-07 at 18 49 45

Steps to Reproduce

  1. Visit https://www.reddit.com/user/meejle
  2. Click the Brave Rewards icon in the address bar

Actual result:

Brave Rewards shows incorrect/unrelated Reddit usernames, and/or usernames seemingly pulled from elsewhere.

Expected result:

Brave Rewards always shows the correct Reddit usernames.

Reproduces how often:

Easily reproduced

Brave version (brave://version info)

Brave 1.24.82 Chromium: 90.0.4430.93 (Official Build) (x86_64)
Revision 4df112c29cfe9a2c69b14195c0275faed4e997a7-refs/branch-heads/4430@{#1348}
OS macOS Version 11.3.1 (Build 20E241)

Version/Channel Information:

  • Can you reproduce this issue with the current release? Yes
  • Can you reproduce this issue with the beta channel? Unknown
  • Can you reproduce this issue with the nightly channel? Unknown

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields? No (nor when disabling AdGuard)
  • 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:

N/A

@VictorieeMan
Copy link

I can confirm that I get the exact same problem independently of your instance. This happened before I found your GitHub issue here, and the description you have given fits my problem perfectly. In other words it replicates.

Just out of interest, have you connected a Twitter account to your Reddit account as well? Because in my case it seems like Brave Rewards try to use my Twitter handle as my Reddit Handle. (But I can't verify the coincidence.)

@meejle
Copy link
Author

meejle commented May 11, 2021

Just out of interest, have you connected a Twitter account to your Reddit account as well? Because in my case it seems like Brave Rewards try to use my Twitter handle as my Reddit Handle. (But I can't verify the coincidence.)

Ahh, that’s really interesting! I hadn’t thought of that. But yes, my Twitter account is connected to my Reddit account.

@VictorieeMan, to narrow it down even more: have you “claimed” your Twitter account on Brave Rewards Creators? Because I guess if you haven’t we at least know it’s Reddit causing that particular issue, and not Brave Rewards!

Out of interest, does the other part of my issue happen to you as well (where the Brave Rewards button shows the same username, no matter whose Reddit profile page you’re on)?

(Note to devs: I’m going to leave my Twitter and Reddit accounts linked in case it helps you diagnose this. Just so you know I haven’t disconnected them to see if it fixes it!)

@VictorieeMan
Copy link

VictorieeMan commented May 11, 2021

Yes I have linked the twitter as well, things over at twitters does work fine though. I've only experienced problems with Brave Rewards over at Reddit.

Interesting suggestion @meejle, I tried it out and got the same result as you mentioned (Clicking other profiles to tip them) and got the following results:

  1. Clicking u/TheVictorieeMan (Tip, myself): https://u.pcloud.link/publink/show?code=XZnldeXZlkhtWbJKhDfSU0aOUN3ylXIQwFNk
  2. Clicking u/meejle (Tipping @meejle): https://u.pcloud.link/publink/show?code=XZDCdeXZxFUal85IBEzg95AXVJyNYLnLSm4X
  3. Clicking a random thrid party user: https://u.pcloud.link/publink/show?code=XZyKdeXZ8gQeYdserJ4hfI5b07skPBMvuVmV

Notice the differences between the results in each case. The first does differ from the second two, although all of them generate the wrong result.

  • All three cases says "VictorieeMan on Reddit", that's wrong for all the cases.
  • All three cases says "Note: This creator... [has not signed up to Brave Rewards]".
  • Case 1. mentions u/VictorieeMan, but ought to be u/TheVictorieeMan
  • Case 2 and 3, mentions the correct u/[handles] for both users but still calls it "VictorieeMan's post".

This seems to be the same thing you mentioned. Thanks for suggesting me to check the same.

@CloggyAbroad
Copy link

This error only seems to happen if I am logged into reddit when I am trying to tip. When I log out the correct reddit username appears in the tipping panel.

@meejle
Copy link
Author

meejle commented May 11, 2021

This error only seems to happen if I am logged into reddit when I am trying to tip. When I log out the correct reddit username appears in the tipping panel.

@CloggyAbroad Interesting! Yep, once I logged out, it at least got the users right... although it still shows mine and @VictorieeMan's Twitter usernames.

Running more scenarios (because I liked how @VictorieeMan did it 😃):

Logged out, visiting https://www.reddit.com/user/TheVictorieeMan
Shows "u/VictorieeMan" (correct user but incorrectly displays his Twitter username)

Logged out, visiting https://www.reddit.com/user/meejle
Shows "u/mikeleewrites" (correct user but incorrectly displays my Twitter username)

Logged in, visiting https://www.reddit.com/user/TheVictorieeMan
Shows "u/thinsoldier" (incorrect user)

Logged in, visiting https://www.reddit.com/user/meejle
Shows "u/thinsoldier" (incorrect user)

@LaurenWags
Copy link
Member

cc @brave/rewards-client

also, I logged #12302 which may or may not be related.

@VictorieeMan
Copy link

This error only seems to happen if I am logged into reddit when I am trying to tip. When I log out the correct reddit username appears in the tipping panel.

@CloggyAbroad Interesting! Yep, once I logged out, it at least got the users right... although it still shows mine and @VictorieeMan's Twitter usernames.

Running more scenarios (because I liked how @VictorieeMan did it 😃):

Logged out, visiting https://www.reddit.com/user/TheVictorieeMan
Shows "u/VictorieeMan" (correct user but incorrectly displays his Twitter username)

Logged out, visiting https://www.reddit.com/user/meejle
Shows "u/mikeleewrites" (correct user but incorrectly displays my Twitter username)

Logged in, visiting https://www.reddit.com/user/TheVictorieeMan
Shows "u/thinsoldier" (incorrect user)

Logged in, visiting https://www.reddit.com/user/meejle
Shows "u/thinsoldier" (incorrect user)

Interesting @meejle, that your logged in visit get the "u/thinsoldier" when visiting me but when visiting logged out you get "u/VictorieeMan". Judging from that you get the same result as me when logged out but a local result when logged in.

And as for your similar issue @LaurenWags, it seems related in the results but the #12302 issue does for some reason have a much longer list of steps to replicate the problem. It's also from October 2020, has there been new releases since then, and if does it replicate on the latest release? Maybe it's gone worse judging from our short list of replicating the problem - haha.

@francois-rozet
Copy link

francois-rozet commented May 30, 2021

The problem likely comes from this file

https://github.com/brave/brave-core/blob/1cb5818aa0b70666c6aeea5ea9c06cc4e712171a/vendor/bat-native-ledger/src/bat/ledger/internal/legacy/media/reddit.cc

which has a few methods to detect users, like GetUserNameFromUrl or GetProfileUrl.

It should not be too hard to test each of them, but I don't know how to compile these files 😞

@McKenneddy
Copy link

So brave recognises me as Fallengreg on reddit (should be Jjauso) ...and i have never had any name remotely close to that ...hope the issue gets fixed

@Miyayes
Copy link
Collaborator

Miyayes commented Jun 9, 2021

@Miyayes Miyayes added the priority/P2 A bad problem. We might uplift this to the next planned release. label Jun 9, 2021
@Miyayes
Copy link
Collaborator

Miyayes commented Jun 10, 2021

@LaurenWags
Copy link
Member

LaurenWags commented Jul 26, 2021

Verified using

Brave	1.27.108 Chromium: 92.0.4515.107 (Official Build) (x86_64)
Revision	87a818b10553a07434ea9e2b6dccf3cbe7895134-refs/branch-heads/4515@{#1634}
OS	macOS Version 10.15.7 (Build 19H1217)

and dev component 1.0.48.

Verified both test plans from brave/brave-site-specific-scripts#59 (comment).
Confirmed I saw the correct Reddit user information whether I was logged in or not from panel, tip banner, and in-line tip banner.

Examples (Logged in):

Example Example Example Example
Example1 Example2 Example3 Example4

Examples (Not logged in):

Example Example Example Example
1 2 4 5

Additionally, I reproduced the issue (when logged in only, which is in line with the comment from #15687 (comment)) with prod component version 1.0.42 and different Reddit users. When I relaunched using 1.0.48 and refreshed the Reddit page (due to #11825), when I opened the panel the correct/expected Reddit creator information was shown.

1.0.42 1.0.48
prod dev

Encountered and logged #17167 while testing this issue.


Verification PASSED on Win 10 x64 using the following build:

Brave | 1.27.109 Chromium: 92.0.4515.115 (Official Build) (64-bit)
--- | ---
Revision | 48cb2f4029b84b003719740a6cf9ca73f374a857-refs/branch-heads/4515_105@{#4}
OS | Windows 10 OS Version 2009 (Build 19042.1110)

Verified both cases outlined via brave/brave-site-specific-scripts#59 (comment) using Brave Local Data Updater - Version: 1.0.49. Confirmed/Verified that the correct reddit user information was being displayed within the rewards panel, tipping banner and in-line tipping banner as per the following:

Logged in Example Logged in Example Logged in Example Logged in Example
image image image image
Logged in Example Logged in Example Logged in Example Logged in Example
image image image image

@VictorieeMan
Copy link

Verified using

Brave	1.27.108 Chromium: 92.0.4515.107 (Official Build) (x86_64)
Revision	87a818b10553a07434ea9e2b6dccf3cbe7895134-refs/branch-heads/4515@{#1634}
OS	macOS Version 10.15.7 (Build 19H1217)

and dev component 1.0.48.

Verified both test plans from brave/brave-site-specific-scripts#59 (comment).
Confirmed I saw the correct Reddit user information whether I was logged in or not from panel, tip banner, and in-line tip banner.

Examples (Logged in):
Example Example Example Example
Example1

Great work with the pictures, it takes some time to produce and properly include screenshots in a comment.

It's a false confirmation, however, since the username of my account is u/TheVictorieeMan and not u/VictorieeMan, which is very confusing and brings doubt about whether the funds reach the correct user. This strange difference between the Reddit username and what the banner shows can be seen within the example screenshots provided.

As far as I understand, the purpose of showing the name u/ in the banner is to confirm that it reaches the correct account. But that confirmation fails if the username on Reddit doesn't match the BAT-banner, even tough it should.

My suggestion is to reopen this problem until it's actually solved, which it isn't.

=== ===

Great job with the involvement though, I find it useful to add some compliments for that. Because it's too easy to seem like such a downer when disproving something.

@LaurenWags
Copy link
Member

@VictorieeMan has your Reddit username ever been VictorieeMan or was it always TheVictorieeMan?

@VictorieeMan
Copy link

@LaurenWags I have never changed my Reddit username from TheVictorieeMan. My theory is that the brave rewards confuses the Reddit name with the twitter handle associated with the Reddit account, which isn't good. The original thread opener [@meejle) hinted about this:

"Strangely enough, there is a u/Mikeleewrites (with a capital "M") on Reddit, but he isn't me. I only have one Reddit account, and it's always been u/meejle. HOWEVER: My Twitter username is @mikeleewrites (with a small "m"). I've claimed that Twitter account on my Brave Rewards Creator profile, so I can only imagine that's where Brave browser is somehow "finding" the "mikeleewrites" (with a small "m") username?"

And the same goes for my account. It looks like it takes the twitter username and presents is in the form of a Reddit username with "u/" instead of "@" making things even more confusing.

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