brave-browser
brave-browser copied to clipboard
[Skus] - SkusSDK does not propagate errors to the point of usage
Description:
- SkusSDK can return
499 - bad request - null
as an error message and we have no idea what error it is. - So we need to propagate errors so we can decide what to do when an error occurs.
- This means errors and results should NOT be returned in the same string. It should explicitly return an error code with the error message upon failure. Otherwise it should return no error code (
Result::Ok
) and the requested data.
Steps to Reproduce
- Attempt to Link accounts from Mobile to Desktop
- Sometimes you'll get an error "Unknown"
Reproduces how often: [Easily reproduced, Intermittent Issue]
- Easily reproduced
Brave Version:
- All
Device details:
- All
Removing from the 1.66.x
milestone as https://github.com/brave/brave-core/pull/22944 needs to be merged into master
and then uplifted into 1.66.x
if needed.
Verification PASSED
using
Brave | 1.69.124 Chromium: 127.0.6533.73 (Official Build) beta (64-bit)
-- | --
Revision | dbc9035349f74e493f3293d93f4cbf35c597dfec
OS | Windows 11 Version 23H2 (Build 22631.3880)
Desktop - Mobile Linking
Desktop - Desktop - Passed
- installed 1.69.124
- launched Brave
- loaded account.bravesoftware.com
- entered basic-auth credentials
- entered [email protected], clicked Get login link
- clicked the link in the email
- clicked Browse plans
- scrolled down and clicked on Buy now for Brave VPN Subscription
- completed the Stripe purchase flow
- connected VPN via VPN panel
- switched to another region, Brazil
- loaded whatismyipaddress.com and confirmed correct region is shown
Machine 1:
step 10 | step 11 | step 12 |
---|---|---|
Machine 2:
- Installed 1.69.125
- launched Brave
- loaded account.bravesoftware.com
- entered basic-auth credentials
- entered [email protected], clicked Get login link
- clicked the link in the email
- clicked Refresh Brave VPN button
- You have Active credentials loaded! page shown
- connected to VPN via VPN panel successfully
- switched to Brazil region
- loaded whatismyipaddress.com and confirmed correct region is shown
step 7 | step 8 | step 9 | step 11 |
---|---|---|---|
Desktop - Android - Failed
- Installed 1.69.125
- launched Brave
- loaded account.bravesoftware.com
- entered basic-auth credentials
- entered [email protected], clicked Get login link
- clicked the link in the email
- clicked Refresh Brave VPN button
- You have Active credentials loaded! page shown
- connected to VPN via VPN panel successfully
- switched to Canada region
- loaded whatismyipaddress.com and confirmed correct region is shown - Failed. Still shown my default region.
example | example | example | example | example |
---|---|---|---|---|
Desktop - iOS - PASSED
Note: tested on Brave nightly from TestFlight Build
1.Installed Brave nightly 1.70.24 2.launched Brave 3. loaded account.bravesoftware.com 4. entered basic-auth credentials 5. entered [email protected], clicked Get login link 6. clicked the link in the email 7. clicked Refresh Brave VPN button 8. You have Active credentials loaded! page shown 9. connected to VPN via VPN panel successfully 10. switched to Brazil region 11. loaded whatismyipaddress.com and confirmed correct region is shown
example | example | example | example | example |
---|---|---|---|---|
Verified on Pixel 7
using version(s):
Device/OS: Pixel 7 / panther_beta-user 15 AP31.240617.009 release-keys
Brave build: 1.69.124
Chromium: 127.0.6533.73 (Official Build) beta (64-bit)
VPN
subscription link from Android
to Desktop
STEPS:
- Launch Brave on
Android
- Purchase
VPN
subscription - Hamburger menu > Settings > Brave Firewall + VPN
- Link purchase to your Brave account
- Enter email > Go to email and tap the confirmation button
- On
account.brave.com
tapLink my subscription now
> Confirm successful result - Open Brave on
Desktop
> Go toaccount.brave.com
- Sign in with the same email > Confirm
You have active credentials loaded!
message is shown - Go to www.ipinfo.io > Change
VPN
location in Brave > Verify results on www.ipinfo.io
ACTUAL RESULTS:
- Verified that purchased on
Android
VPN
subscription can be linked toDesktop
- Verified that changing the
VPN
location in Brave is working and shows the correct location on www.ipinfo.io
https://github.com/user-attachments/assets/6f3e0d8e-74c3-41a2-b730-072834ed927f
Verified on iPhone 14
using version(s):
Device/OS: iPhone 14 / iOS 17.6.1
Brave build: 1.69 (162)
BraveCore: 1.69.162 (128.0.6613.120)
Filed follow-up issue #40860
VPN
subscription link from iOS
to Desktop
STEPS:
- Launch Brave on
iOS
- Purchase
VPN
subscription - Hamburger menu > Settings > Brave Firewall + VPN
- Link purchase to your Brave account
- Enter email > Go to email and tap the confirmation button
- On
account.brave.com
tapLink my subscription now
> Confirm successful result - Open Brave on
Desktop
> Go toaccount.brave.com
- Sign in with the same email > Confirm
You have active credentials loaded!
message is shown - Go to www.ipinfo.io > Change
VPN
location in Brave > Verify results on www.ipinfo.io
ACTUAL RESULTS:
- Verify that purchased on
iOS
VPN
subscription can be linked toDesktop
- Observed an error where an active
VPN
subscription can't be linked to theWeb
https://github.com/user-attachments/assets/4578fb4f-818f-4ff1-bb6d-caf22c28477e