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

Android 1.24.x can't retrieve Referral codes from google play services #15903

Closed
kkuehlz opened this issue May 17, 2021 · 1 comment · Fixed by brave/brave-core#8843
Closed
Assignees
Labels
OS/Android Fixes related to Android browser functionality priority/P1 A very extremely bad problem. We might push a hotfix for it. QA Pass - Android ARM QA Pass - Android Tab QA Pass - Android x86 QA/Yes release/blocking release-notes/include

Comments

@kkuehlz
Copy link
Contributor

kkuehlz commented May 17, 2021

Description

Android 1.24.x can't retrieve Referral codes from google play services. According to @samartnik, this is due to a change in the abb format.

Steps to reproduce

  1. Uninstall Brave from Android.
  2. Navigate to brave.com/?ref=TVT141 in Chrome or some other browser
  3. Use the link to download brave
  4. Check brave://local-state

Actual result

download_id missing. referral_code is BRV001 (default)

Expected result

download_id should be present. referral_code should be TVT141

Issue reproduces how often

Always

Version/Channel Information:

1.24.x

  • Can you reproduce this issue with the current Play Store version?
    y
  • Can you reproduce this issue with the current Play Store Beta version?
    n/a
  • Can you reproduce this issue with the current Play Store Nightly version?
    n/a

Device details

  • Install type (ARM, x86):
  • Device type (Phone, Tablet, Phablet):
  • Android version:

Brave version

1.24.85

@kkuehlz kkuehlz added release/blocking priority/P1 A very extremely bad problem. We might push a hotfix for it. OS/Android Fixes related to Android browser functionality labels May 17, 2021
@kkuehlz kkuehlz added this to the 1.24.x - Release #4 milestone May 17, 2021
@samartnik samartnik self-assigned this May 18, 2021
@kjozwiak
Copy link
Member

kjozwiak commented May 20, 2021

Verification PASSED on Samsung S10+ running Android 11 using 1.24.86 Chromium: 90.0.4430.212

"referral": {
         "download_id": "e93fb9f7-2d1e-41e4-b573-f1490ed4140c",
         "headers": [ {
            "cookieNames": [  ],
            "domains": [ "eaff.com", "stg.eaff.com" ],
            "expiration": 31536000000.0,
            "headers": {
               "X-Brave-Partner": "eaff"
            }
         }, {
            "cookieNames": [  ],
            "domains": [ "sandbox.uphold.com", "api-sandbox.uphold.com", "uphold.com", "api.uphold.com" ],
            "expiration": 31536000000.0,
            "headers": {
               "X-Brave-Partner": "uphold"
            }
         }, {
            "cookieNames": [  ],
            "domains": [ "www.grammarly.com", "grammarly.com", "static.grammarly.com", "gnar.grammarly.com" ],
            "expiration": 31536000000.0,
            "headers": {
               "X-Brave-Partner": "grammarly"
            }
         } ],
         "initialization": true,
         "promo_code": "TVT141"
      },
      "referral_android_first_run_timestamp": "13265978169564063",
      "stats": {
         "first_check_made": true,
         "last_check_month": 5,
         "last_check_woy": 20,
         "last_check_ymd": "2021-05-20",
         "threshold_query": "https://laptop-updates.brave.com/1/usage/brave-core-threshold?platform=android-bc&channel=release&version=1.24.86&daily=true&weekly=true&monthly=true&first=true&woi=2021-05-17&dtoi=2021-05-20&ref=TVT141&adsEnabled=false&arch=",
         "week_of_installation": "2021-05-17"
      }

Screenshot_20210520-055639_Brave


Verification passed on Samsung Tab A with Android 10 running 1.24.86 x64 build

referral": {
         "download_id": "d3deb732-8065-4976-b070-3d88ab71f519",
         "headers": [ {
            "cookieNames": [  ],
            "domains": [ "eaff.com", "stg.eaff.com" ],
            "expiration": 31536000000.0,
            "headers": {
               "X-Brave-Partner": "eaff"
            }
         }, {
            "cookieNames": [  ],
            "domains": [ "sandbox.uphold.com", "api-sandbox.uphold.com", "uphold.com", "api.uphold.com" ],
            "expiration": 31536000000.0,
            "headers": {
               "X-Brave-Partner": "uphold"
            }
         }, {
            "cookieNames": [  ],
            "domains": [ "www.grammarly.com", "grammarly.com", "static.grammarly.com", "gnar.grammarly.com" ],
            "expiration": 31536000000.0,
            "headers": {
               "X-Brave-Partner": "grammarly"
            }
         } ],
         "initialization": true,
         "promo_code": "TVT141"
      },
      "referral_android_first_run_timestamp": "13265979012596593",
      "stats": {
         "first_check_made": true,
         "last_check_month": 5,
         "last_check_woy": 20,
         "last_check_ymd": "2021-05-20",
         "threshold_query": "https://laptop-updates.brave.com/1/usage/brave-core-threshold?platform=android-bc&channel=release&version=1.24.86&daily=true&weekly=true&monthly=true&first=true&woi=2021-05-17&dtoi=2021-05-20&ref=TVT141&adsEnabled=false&arch=",
         "week_of_installation": "2021-05-17"
      }
   },

image


Verification PASSED on Asus Zenfone (x86) running Android 6 using 1.24.86 Chromium: 90.0.4430.212

      "referral": {
         "download_id": "be055180-87b9-4632-b057-d8eb69063b34",
         "headers": [ {
            "cookieNames": [  ],
            "domains": [ "eaff.com", "stg.eaff.com" ],
            "expiration": 31536000000.0,
            "headers": {
               "X-Brave-Partner": "eaff"
            }
         }, {
            "cookieNames": [  ],
            "domains": [ "sandbox.uphold.com", "api-sandbox.uphold.com", "uphold.com", "api.uphold.com" ],
            "expiration": 31536000000.0,
            "headers": {
               "X-Brave-Partner": "uphold"
            }
         }, {
            "cookieNames": [  ],
            "domains": [ "www.grammarly.com", "grammarly.com", "static.grammarly.com", "gnar.grammarly.com" ],
            "expiration": 31536000000.0,
            "headers": {
               "X-Brave-Partner": "grammarly"
            }
         } ],
         "initialization": true,
         "promo_code": "TVT141"
      },

x86

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OS/Android Fixes related to Android browser functionality priority/P1 A very extremely bad problem. We might push a hotfix for it. QA Pass - Android ARM QA Pass - Android Tab QA Pass - Android x86 QA/Yes release/blocking release-notes/include
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants