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

5% fee not always sent #7425

Closed
LaurenWags opened this issue Dec 17, 2019 · 4 comments · Fixed by brave/brave-core#4369
Closed

5% fee not always sent #7425

LaurenWags opened this issue Dec 17, 2019 · 4 comments · Fixed by brave/brave-core#4369

Comments

@LaurenWags
Copy link
Member

Description

If you tip several sites in quick succession, before the 5% is deducted for each one, the 5% will not be sent. 5% is only sent for the last site tipped.

Steps to Reproduce

  1. Clean profile on staging - I launched with --rewards=staging=true --brave-ads-staging --enable-logging=stderr --vmodule=*rewards*=6 --log-level=2
  2. Enable Rewards (don't claim grant)
  3. Connect your verified, KYC'd, staging account. Be sure you have enough BAT to tip several publishers.
  4. Tip a verified, KYC'd publisher once (I used 3zsistemi.si)
  5. Navigate to Uphold and confirm on your Uphold Activity account page that both the tip and 5% fee were sent (see A in screenshot).
  6. Open 3-4 verified, KYC'd publisher sites in tabs. (I uses laurenwags.github.io, vimeo.com/bravelaurenwags, twitter.com/bravelaurenwags, github.com/laurenwags).
  7. Tip one, move quickly to the next, tip again, repeat until you've tipped all 3-4 publishers from step 6.
  8. Go to Uphold, open the Activity page for your Brave Browser card and confirm that all tips (not the 5%) were made.
  9. Wait a few minutes. Only the 5% for the last tip sent will be posted to this page. Even if you wait more time, the additional 5% fees are not sent.

Actual result:

Screen Shot 2019-12-17 at 9 48 23 AM

Expected result:

5% fees to be sent even when tipping in rapid succession

Reproduces how often:

easily

Brave version (brave://version info)

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

Version/Channel Information:

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

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:

Reproduced several times
cc @NejcZdovc @brave/legacy_qa

@NejcZdovc NejcZdovc added the priority/P1 A very extremely bad problem. We might push a hotfix for it. label Dec 17, 2019
@NejcZdovc NejcZdovc added this to the 1.2.x - Beta milestone Dec 18, 2019
@NejcZdovc NejcZdovc self-assigned this Dec 18, 2019
@kjozwiak
Copy link
Member

kjozwiak commented Jan 3, 2020

@NejcZdovc @rebron any updates on this one? It's labelled as a P1 and was moved into the 1.2.x milestone. We'll probably want to get this fixed before the Jan 6th release.

@rebron rebron removed this from the 1.2.x - Release milestone Jan 3, 2020
@rebron
Copy link
Collaborator

rebron commented Jan 3, 2020

Removed from 1.2.x milestone. Need to investigate this one a bit more.

@masparrow
Copy link

Verified the behaviour on Mac OSX - sending 4 tips in quick succession, only one 5% BAT contribution was made. I took records of the Uphold activity over the space of an hour;

Screenshot 2020-01-13 at 15 04 44

Now investigating the cause.

masparrow added a commit to brave/brave-core that referenced this issue Jan 15, 2020
Resolves brave/brave-browser#7425

Browser test

Update
@NejcZdovc NejcZdovc added this to the 1.5.x - Nightly milestone Jan 21, 2020
@LaurenWags
Copy link
Member Author

LaurenWags commented Mar 3, 2020

Verified passed with

Brave 1.5.106 Chromium: 80.0.3987.122 (Official Build) beta (64-bit)
Revision cf72c4c4f7db75bc3da689cd76513962d31c7b52-refs/branch-heads/3987@{#943}
OS macOS Version 10.14.6 (Build 18G3020)
  • Verified STR from description

Screen Shot 2020-03-03 at 11 54 17 AM

Verification passed on

Brave 1.5.108 Chromium: 80.0.3987.132 (Official Build) beta (64-bit)
Revision fcea73228632975e052eb90fcf6cd1752d3b42b4-refs/branch-heads/3987@{#974}
OS Windows 10 OS Version 1803 (Build 17134.1006)
  • Verified STR from the description

image

Verification passed on

Brave 1.5.109 Chromium: 80.0.3987.132 (Official Build) beta (64-bit)
Revision fcea73228632975e052eb90fcf6cd1752d3b42b4-refs/branch-heads/3987@{#974}
OS Ubuntu 18.04 LTS

Verified the test plan from the description

image

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.

7 participants