Skip to content
This repository was archived by the owner on Feb 24, 2025. It is now read-only.

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 subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants