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

Incorrect state of VPN button when user refreshes credentials #30881

Closed
srirambv opened this issue Jun 7, 2023 · 3 comments · Fixed by brave/brave-core#18842
Closed

Incorrect state of VPN button when user refreshes credentials #30881

srirambv opened this issue Jun 7, 2023 · 3 comments · Fixed by brave/brave-core#18842

Comments

@srirambv
Copy link
Contributor

srirambv commented Jun 7, 2023

Description

Incorrect state of VPN button when user refreshes credentials

Steps to Reproduce

  1. Refresh VPN credentials from https://account.brave.com
  2. Wait for the Panel to load the VPN state
  3. Button is disabled but VPN is enabled
  4. If you have multiple profiles it shows correctly when launching other profile

Actual result:

image
image

Expected result:

Should show correct state

Reproduces how often:

Easy

Brave version (brave://version info)

Brave 1.54.38 Chromium: 114.0.5735.90 (Official Build) nightly (64-bit)
Revision 386bc09e8f4f2e025eddae123f36f6263096ae49-refs/branch-heads/5735@{#1052}
OS Windows 11 Version 22H2 (Build 22621.1778)

Version/Channel Information:

  • Can you reproduce this issue with the current release? Maybe
  • Can you reproduce this issue with the beta channel? Maybe
  • Can you reproduce this issue with the nightly channel? Yes

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields? NA
  • Does the issue resolve itself when disabling Brave Rewards? NA
  • Is the issue reproducible on the latest version of Chrome? NA

Miscellaneous Information:

cc: @bsclifton @brave/qa-team

@simonhong
Copy link
Member

simonhong commented Jun 9, 2023

Trying to repro.

I can repro with below steps.

  1. Disconnect vpn from panel
  2. Reset skus state from brave://skus-internals
  3. Connect vpn from os vpn settings
  4. Load account.brave.com and refresh
  5. Launch panel and panel shows as connected but button isn't

Panel gets current connected state when it's launched.
However, toolbar button update its state when it gets connection state change noti.
Before purchased state, os connection api is in already connected state as connected via os vpn settings.
but BraveVpnService doesn't notify this to observers as it's not purchased state.(BraveVpnService::OnConnectionStateChanged())
So, button still don't know current connected state.

simonhong added a commit to brave/brave-core that referenced this issue Jun 9, 2023
simonhong added a commit to brave/brave-core that referenced this issue Jun 9, 2023
simonhong added a commit to brave/brave-core that referenced this issue Jun 9, 2023
simonhong added a commit to brave/brave-core that referenced this issue Jun 9, 2023
@simonhong simonhong added this to In Progress in VPN Jun 9, 2023
VPN automation moved this from In Progress to Completed Jun 9, 2023
@brave-builds brave-builds added this to the 1.54.x - Nightly milestone Jun 9, 2023
@MadhaviSeelam
Copy link
Collaborator

MadhaviSeelam commented Jul 6, 2023

Verification PASSED using

Brave | 1.54.91 Chromium: 115.0.5790.56 (Official Build) beta (64-bit)
-- | --
Revision | ebcfb87c1393f36955cfa2f6451a1767e3dc3ee5-refs/branch-heads/5790@{#1193}
OS | Windows 11 Version 22H2 (Build 22621.1848)

Verified using STR from #30881 (comment) & #30881 (comment)

Confirmed VPN button shows correct state (with green check mark ✅) when VPN is connected via VPN panel & OS settings

Confirmed VPN button shows correct state (with gray minus ➖) if VPN is disconnected

Confirmed VPN button shows correct state (with red VPN button) if VPN connection failed

Connected Disconnected Failed
connected connected connected

@stephendonner
Copy link
Collaborator

stephendonner commented Aug 4, 2023

Verification PASSED using

Brave 1.57.36 Chromium: 115.0.5790.171 (Official Build) beta (x86_64)
Revision a4bb683f14dc402f5da13707e13559dc4d5ef08b
OS macOS Version 14.0 (Build 23A5301h)

Shared Steps:

  1. install 1.57.36
  2. launched Brave
  3. loaded account.bravesoftware.com
  4. purchased, configured, and connected to Brave VPN
  5. confirmed I was connected to Brave VPN
  6. toggled Brave VPN to Disconnected
  7. confirmed I was Disconnected

Connected

Screenshot 2023-08-04 at 12 47 39 PM

Disconnected

Screenshot 2023-08-04 at 12 47 15 PM

Can't connect to server

Screenshot 2023-08-04 at 12 48 18 PM

(continued from Shared Steps)

  1. disabled Wi-Fi
  2. disconnected from Brave VPN
  3. tried to reconnect by clicking on the VPN icon

SKUS reset / Start free trial

Screenshot 2023-08-04 at 12 50 26 PM

(continued from Shared Steps)

  1. opened brave://skus-internals
  2. clicked on Reset SKUs state
  3. shut down Brave
  4. launched Brave
  5. clicked on the VPN button in the toolbar
Connected Disconnected Can't connect to server Start free trial
Screenshot 2023-08-04 at 11 29 40 AM Screenshot 2023-08-04 at 11 29 52 AM Screenshot 2023-08-04 at 12 36 59 PM Screenshot 2023-08-04 at 12 24 04 PM

Encountered and logged:

@stephendonner stephendonner added QA/In-Progress Indicates that QA is currently in progress for that particular issue QA Pass-macOS and removed QA/In-Progress Indicates that QA is currently in progress for that particular issue labels Aug 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment