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

Reset Split tunneling when config is reset #25010

Closed
srirambv opened this issue Aug 26, 2022 · 1 comment · Fixed by brave/brave-core#14999
Closed

Reset Split tunneling when config is reset #25010

srirambv opened this issue Aug 26, 2022 · 1 comment · Fixed by brave/brave-core#14999

Comments

@srirambv
Copy link
Contributor

Description

Reset Split tunneling when config is reset

Steps to reproduce

  1. Enable VPN on nightly
  2. Enable Split tunneling for specific Apps
  3. Reset VPN config via VPN settings in menu
  4. Split tunneling is not reset

Actual result

Split tunneling is not reset when config is reset

Expected result

Reset Split tunneling when config is reset

Issue reproduces how often

Easy

Version/Channel Information:

  • Can you reproduce this issue with the current Play Store version? NA
  • Can you reproduce this issue with the current Play Store Beta version? NA
  • Can you reproduce this issue with the current Play Store Nightly version? Yes

Device details

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

Brave version

1.45.2

Website problems only

  • 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

Additional information

cc: @timchilds @deeppandya @brave/legacy_qa

@GeetaSarvadnya
Copy link

Verification PASSED on vivo x70 Pro with Android version 12 running Bravemonoarn64.apk_1.45.106

  • Verified the STR from the description and ensured Split tunneling is resets when VPN config is reset
Example Example
Screenshot_20221019_175420 Screenshot_20221019_175518

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

Successfully merging a pull request may close this issue.

4 participants