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

can't accept grants due to "statusCode":400,"error":"Bad Request","message":"Invalid request payload input" #7443

Closed
kjozwiak opened this issue Dec 18, 2019 · 2 comments

Comments

@kjozwiak
Copy link
Member

kjozwiak commented Dec 18, 2019

Description

Looks like Release & Beta have regressed and users can't accept grants and are getting the Ooops, something is wrong. Please try again later. I've reproduced the issue on staging for both UGP & Ad grants.

@evq took a look at the backend and mentioned the issue is most likely client based. Sounds like the client is passing the wrong format to the backend when attempting to accept grants.

There's also been several reports via the community and internally as well.

Steps to Reproduce

Test Case #1

  1. launch 1.1.22 CR: 79.0.3945.7 using --enable-logging=stderr --rewards=staging=true
  2. enable rewards via the panel or brave://rewards
  3. attempt to accept the UGP grant on staging

Test Case #2

  1. launch 1.1.22 CR: 79.0.3945.7 using --enable-logging=stderr --rewards=staging=true
  2. enable rewards via the panel or brave://rewards
  3. restore an internal wallet that has a ad grant and attempt to accept it

Actual result:

Screen Shot 2019-12-17 at 8 31 18 PM

[ RESPONSE - SetGrantCallback ]
> time: 1576631356
> result: Failure
> response: {"statusCode":400,"error":"Bad Request","message":"Invalid request payload input"}
> headers accept-ranges: bytes
> headers cache-control: no-cache
> headers content-length: 82
> headers content-type: application/json; charset=utf-8
> headers date: Wed, 18 Dec 2019 01:09:16 GMT
> headers server: Cowboy
> headers status: 400
> headers via: 1.1 varnish
> headers x-cache: MISS
> headers x-cache-hits: 0
> headers x-served-by: cache-yyz4532-YYZ
[ END RESPONSE ]

Expected result:

Users should be able to accept grants without any issues.

Reproduces how often:

100% reproducible when using the above STR.

Brave version (brave://version info)

  • 1.1.22 CR: 79.0.3945.79 (Release) - Reproduced
  • 1.2.28 CR: 79.0.3945.79 (Beta) - Reproduced
  • 1.3.67 CR: 79.0.3945.79 (Dev) - Couldn't reproduce
  • 1.4.25 CR: 79.0.3945.79 (Nightly) - Couldn't reproduce

Version/Channel Information:

  • Can you reproduce this issue with the current release? Yes
  • Can you reproduce this issue with the beta channel? Yes
  • Can you reproduce this issue with the dev channel? Yes
  • 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 @bsclifton @LaurenWags @mandar-brave

@kjozwiak kjozwiak added bug feature/rewards priority/P1 A very extremely bad problem. We might push a hotfix for it. QA/Yes QA/Test-Plan-Specified labels Dec 18, 2019
@kjozwiak kjozwiak added this to the 1.1.x - Release Hotfix 3 milestone Dec 18, 2019
NejcZdovc added a commit to brave/brave-core that referenced this issue Dec 18, 2019
@NejcZdovc NejcZdovc self-assigned this Dec 18, 2019
NejcZdovc added a commit to brave/brave-core that referenced this issue Dec 18, 2019
@kjozwiak
Copy link
Member Author

This was fixed via a server side fix as discussed in Slack 👍

@kjozwiak
Copy link
Member Author

kjozwiak commented Dec 18, 2019

Verification PASSED on macOS 10.15.1 x64 using the following:

  • 1.1.22 CR: 79.0.3945.79 (release) - Claimed both UGP & Ad grant on staging without issues
  • 1.2.28 CR: 79.0.3945.79 (Beta) - Claimed both UGP & Ad grant on staging without issues
  • 1.3.67 CR: 79.0.3945.79 (Dev) - Claimed both UGP & Ad grant on staging without issues
  • 1.4.25 CR: 79.0.3945.79 (Nightly) - Claimed both UGP & Ad grant on staging without issues

Verified passed with macOS 10.13.6 x64 using

  • 1.1.22 CR: 79.0.3945.79 (release) - Claimed both UGP & Ad grant on staging without issues
  • 1.2.29 CR: 79.0.3945.79 (Beta) - Claimed UGP grant on staging without issue

Verification passed on

Brave 1.1.23 Chromium: 79.0.3945.88 (Official Build) (64-bit)
Revision c2a58a36b9411c80829b4b154bfcab97e581f1f3-refs/branch-heads/3945@{#954}
OS Ubuntu 18.04 LTS

Verification passed on

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

Verification passed on

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

Verification passed on

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

Verification PASSED on Win 10 x64 using the following:

  • 1.1.23 CR: 79.0.3945.88 (release) - Claimed both UGP & Ad grant on staging without issues
  • 1.2.30 CR: 79.0.3945.79 (Beta) - Claimed both UGP & Ad grant on staging without issues
  • 1.3.68 CR: 79.0.3945.79 (Dev) - Claimed both UGP & Ad grant on staging without issues
  • 1.4.38 CR: 79.0.3945.88 (Nightly) - Claimed both UGP & Ad grant on staging without issues

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

No branches or pull requests

4 participants