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

Rename BraveVpnServiceName to BraveVpnHelperName #29820

Closed
spylogsster opened this issue Apr 18, 2023 · 4 comments · Fixed by brave/brave-core#18122
Closed

Rename BraveVpnServiceName to BraveVpnHelperName #29820

spylogsster opened this issue Apr 18, 2023 · 4 comments · Fixed by brave/brave-core#18122

Comments

@spylogsster
Copy link

In order to prepare to land implementation of #29253 we need to do some refactorings. BraveVpnHelper is used as BraveVpnService sometimes and would be better to name it as BraveVpnHelper evrywhere

@stephendonner
Copy link

stephendonner commented May 12, 2023

Since I'm blocked on Windows 10 by #30291 (thx @MadhaviSeelam !), will do a quick macOS end-to-end test, just as a regression check 👍

@stephendonner
Copy link

stephendonner commented May 12, 2023

Verified PASSED using

Brave 1.52.94 Chromium: 113.0.5672.92 (Official Build) beta (x86_64)
Revision b6f521170062a1fa8a82c33fb223b06fec566da1-refs/branch-heads/5672_63@{#10}
OS macOS Version 11.7.6 (Build 20G1231)

Steps:

  1. installed 1.52.94
  2. launched Brave
  3. using account.bravesoftware.com, purchased, configured, and connected to Brave VPN
  4. changed regions to Brazil
  5. loaded browserleaks.com/dns and confirmed no IPv4 or IPv6 DNS-server IP address leaks

Confirmed I could purchase, configure, and connect to Brave VPN

example example example example
Screen Shot 2023-05-11 at 5 12 12 PM Screen Shot 2023-05-11 at 5 14 00 PM Screen Shot 2023-05-11 at 5 14 06 PM Screen Shot 2023-05-11 at 5 14 33 PM

@stephendonner stephendonner added QA Pass-macOS QA/In-Progress Indicates that QA is currently in progress for that particular issue and removed QA/In-Progress Indicates that QA is currently in progress for that particular issue labels May 12, 2023
@stephendonner
Copy link

stephendonner commented May 15, 2023

Verification PASSED using

Brave 1.52.99 Chromium: 113.0.5672.92 (Official Build) beta (64-bit)
Revision b6f521170062a1fa8a82c33fb223b06fec566da1-refs/branch-heads/5672_63@{#10}
OS Windows 10 Version 22H2 (Build 19045.2965)

Case 1: Brave VPN helper service auto-start - PASSED

Steps:

  1. installed 1.52.99
  2. launched Brave
  3. purchased, configured, and connected to Brave VPN
  4. pressed ctrl + alt + del to open Task Manager
  5. confirmed Brave VPN Helper entry in the above
  6. confirmed brave_vpn_helper.exe is running, with an additional process to catch its crashes
  7. shut down Brave and disconnected from Brave VPN
  8. restarted Brave but did NOT connect to Brave VPN

Confirmed brave_vpn_helper.exe service is auto-started when connected to VPN, and is stopped/shut down correctly on both app shut down and app restart

Brave VPN helper dialog brave_vpn_helper.exe Brave, shut down Brave, running, not connected
image (16) image (7) image image

Case 2: Brave VPN helper service restart - PASSED

Steps:

(continued from Case 1)

  1. with Brave running, connected to Brave VPN
  2. opened Task Manager
  3. confirmed brave_vpn_helper.exe is running
  4. selected the helper process
  5. clicked on End task
  6. confirmed brave_vpn_helper.exe restarted
  7. killed the process two more times the same way

Confirmed after killing the helper service, it auto-restarted thrice, and did NOT auto-restart after the 3rd kill

Brave running, w/helper service Brave running, NO helper service
image (7) image

Case 3: Use Secure DNS settings auto-lock - PASSED

Steps:

  1. ran Case 1
  2. opened brave://settings/appearance
  3. noted the state of Use Secure DNS
  4. confirmed in normal operations, with Brave VPN helper service running, Use Secure DNS is checked and the controls are enabled/unlocked
  5. ran Case 2
  6. opened brave://settings/security
  7. confirmed that when Brave VPN helper service is not running any longer, Use Secure DNS is checked but its controls are disabled/locked
Use Secure DNS, unlocked Use Secure DNS, locked
image (13) image (15)

Case 4: No DNS leaks - PASSED

Steps:

  1. ran Case 1
  2. loaded browserleaks.com/dns
  3. noted no DNS IP-address leaks from my ISP
  4. ran Case 2
  5. loaded browserleaks.com/dns
  6. noted no DNS IP-address leaks from my ISP
example example
image image

@stephendonner stephendonner removed the QA/In-Progress Indicates that QA is currently in progress for that particular issue label May 15, 2023
@MadhaviSeelam
Copy link

MadhaviSeelam commented May 20, 2023

Verification PASSED using

Brave | 1.52.102 Chromium: 113.0.5672.126 (Official Build) beta (64-bit)
-- | --
Revision | c541687b21a73452ab403e2dced7033ddc97ee9d-refs/branch-heads/5672@{#1202}
OS | Windows 11 Version 22H2 (Build 22621.1702)

Case 1: Brave VPN helper service auto-start - PASSED

Steps:

  1. installed 1.52.102
  2. launched Brave
  3. purchased, configured, and connected to Brave VPN
  4. pressed ctrl + alt + del to open Task Manager
  5. confirmed Brave VPN Helper entry in the above
  6. confirmed brave_vpn_helper.exe is running, with an additional process to catch its crashes
  7. disconnected from Brave VPN and close Brave
  8. restarted Brave but did NOT connect to Brave VPN
  9. verified Brave VPN Helper entry is not shown
  10. Connected to VPN

Confirmed brave_vpn_helper.exe service is auto-started when connected to VPN, and is stopped/shut down correctly on both app shut down and app restart

VPN panel not connected brave_vpn_helper.exe not shown brave_vpn_helper.exe brave shutdown Brave, running, not connected
image image image image image

Case 2: Brave VPN helper service restart - PASSED

Steps:

(continued from Case 1)

  1. with Brave running, connected to Brave VPN
  2. opened Task Manager
  3. confirmed brave_vpn_helper.exe is running
  4. selected the helper process
  5. clicked on End task
  6. confirmed brave_vpn_helper.exe restarted
  7. killed the process two more times the same way

Confirmed after killing the helper service, it auto-restarted thrice, and did NOT auto-restart after the 3rd kill

Brave running, w/helper service Step 5 Brave running, NO helper service
image image image

Case 3: Use Secure DNS settings auto-lock - PASSED

Steps:

  1. ran Case 1
  2. opened brave://settings/appearance
  3. noted the state of Use Secure DNS
  4. confirmed in normal operations, with Brave VPN helper service running, Use Secure DNS is checked and the controls are enabled/unlocked
  5. ran Case 2
  6. opened brave://settings/security
  7. confirmed that when Brave VPN helper service is not running any longer, Use Secure DNS is checked but its controls are disabled/locked
Use Secure DNS, unlocked Use Secure DNS, locked
image image

Case 4: No DNS leaks - PASSED

Steps:

  1. ran Case 1
  2. loaded browserleaks.com/dns
  3. noted no DNS IP-address leaks from my ISP
  4. ran Case 2
  5. loaded browserleaks.com/dns
  6. noted no DNS IP-address leaks from my ISP
example example
image image

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