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

VPN is sometimes stopped twice #3829

Merged
merged 5 commits into from
Feb 7, 2025
Merged

Conversation

diegoreymendez
Copy link
Contributor

@diegoreymendez diegoreymendez commented Feb 5, 2025

Task/Issue URL: https://app.asana.com/0/1207603085593419/1209322889682604/f

iOS PR: duckduckgo/iOS#3928
BSK PR: duckduckgo/BrowserServicesKit#1213

Description:

Fixes an issue where the tunnel is being stopped twice.

Testing

The same step can be used to reproduce the original issue and test the fix.

  1. Throw an error here.
  2. Verify that there's no yellow alert telling you the wireguard adapter is already started.

Definition of Done:


Internal references:

Pull Request Review Checklist
Software Engineering Expectations
Technical Design Template
Pull Request Documentation

diegoreymendez added a commit to duckduckgo/BrowserServicesKit that referenced this pull request Feb 7, 2025
Task/Issue URL:
https://app.asana.com/0/1207603085593419/1209322889682604/f

iOS PR: duckduckgo/iOS#3928
macOS PR: duckduckgo/macos-browser#3829

What kind of version bump will this require?: Patch

## Description

Fixes an issue where the tunnel is being stopped twice.
diegoreymendez added a commit to duckduckgo/iOS that referenced this pull request Feb 7, 2025
Task/Issue URL:
https://app.asana.com/0/1207603085593419/1209322889682604/f

macOS PR: duckduckgo/macos-browser#3829
BSK PR: duckduckgo/BrowserServicesKit#1213

**Description**:

Fixes an issue where the tunnel is being stopped twice.
@diegoreymendez diegoreymendez merged commit bb86771 into main Feb 7, 2025
20 checks passed
@diegoreymendez diegoreymendez deleted the diego/fix-tunnel-stop-issues branch February 7, 2025 10:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants