-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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 VPN: Unable to load credentials/HTTP 500
error returned from endpoint, after first purchasing Leo Premium
#36321
Comments
HTTP 500
error returned from endpoint HTTP 500
error returned from endpoint, after first purchasing Leo Premium
This insidious "issue" stemmed from me leaving a prior test Leaving open for further diagnosis + followup; potentially, we can look into replacing bogus credentials from past configs, and refetch from Guardian, etc.? |
@evq @nvonpentz @bsclifton while verifying the issue #35933, encountered above issue ( cc: @kjozwiak @LaurenWags
|
I've asked that we prioritize getting to the bottom of https://github.com/brave/account-brave-com/issues/129 before we do more regression testing, esp. since there are likely to be more fixes/changes due to the above's resolution 👍 @MadhaviSeelam @brave/qa-team just FYI |
let's check with @AlanBreck about whether the backend and frontend changes that were made to support this are in an environment where you can test them |
Fixes brave/brave-browser#36321 This change adds product check into `LoadPurchasedState` and `OnCredentialSummary`. Comments around credential usage were updated to be more clear. The call to BraveVpnService::LoadPurchasedState was originally added to `components/skus/renderer/skus_js_handler.cc` so the VPN Service could update its environment (dev, staging, prod) if the person using Brave was switching between brave.software / bravesoftware.com / etc. More info about that and a proposal to create a VPN-specific CLI arg can be found here: brave/brave-browser#38883
added |
The above requires |
@bsclifton does this change touch any Android related code? Should Android be checked as well? |
@LaurenWags sorry for the long response time - yes, technically Android does hit this path too. However, I personally wouldn't test on Android unless you have extra cycles (optional IMO). Testing on Desktop covers the main case and Android has the same code branches when buying from account.brave.com (instead of using in-app-purchase). |
thanks @bsclifton! |
Verification
|
example | example | example | example |
---|---|---|---|
ActiveCredentials
in brave://skus-internals
Leo |
VPN |
---|---|
New Account - purchase VPN
, then Leo Premium
- PASSED
Steps:
- installed
1.68.88
- launched Brave using
--enable-logging=stderr --v=1 --env-leo=staging --env-ai-chat.bsg=dev --env-ai-chat-premium.bsg=dev
- loaded
account.bravesoftware.com
- purchased
Brave VPN
usingjune14-2@mailinator.com
- confirmed I connected to a
Brave VPN
USA (West)
node, usingwhatismyipaddress.com
- purchased
Leo Premium
usingjune14-2@mailinator.com
- confirmed I could
Summarize this page
overPremium
Confirmed ActiveCredentials
for both, using brave://skus-internals
example | example | example | example |
---|---|---|---|
ActiveCredentials
in brave://skus-internals
VPN |
Leo |
---|---|
Description
Brave VPN: Unable to load credentials/
HTTP 500
error returned from endpoint, after first purchasingLeo Premium
Test Plan / Steps to Reproduce
1.65.28
prod
orstaging
account sites; confirm they workBrave VPN
by clicking onStart free trial
Stripe
-purchase/checkout flowYou have active credentials loaded!
green bannerVPN
button on the Brave-browser toolbarActual result:
Unable to load credentials!
error messagingFrom log snippets and @evq it seems clear it's using cached credentials and failing
There's also
HTTP 500
s returned from the credential-provisioning endpoint forBrave VPN
(appears to be on the Guardian side), with logging enabled:Expected result:
Loaded
Brave VPN
credentialsReproduces how often:
100%
Brave version (brave://version info)
Version/Channel Information:
No
No
Yes
Definitely a regression, I just don't-yet have a range nailed down, sorry!
cc @bbondy @mattmcalister @bsclifton @simonhong @brave/qa-team
The text was updated successfully, but these errors were encountered: