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

Remove WireGuard from BSK #958

Merged
merged 6 commits into from
Aug 28, 2024
Merged

Remove WireGuard from BSK #958

merged 6 commits into from
Aug 28, 2024

Conversation

samsymons
Copy link
Contributor

@samsymons samsymons commented Aug 23, 2024

Please review the release process for BrowserServicesKit here.

Required:

Task/Issue URL: https://app.asana.com/0/414709148257752/1207874749913488/f
iOS PR: duckduckgo/iOS#3273
macOS PR: duckduckgo/macos-browser#3144
What kind of version bump will this require?: Major

Description:

This PR removes wireguard-apple as a direct dependency of BSK, and shifts that dependency to the clients.

Steps to test this PR:

  1. See client PRs

OS Testing:

  • iOS 14
  • iOS 15
  • iOS 16
  • macOS 10.15
  • macOS 11
  • macOS 12

Internal references:

Software Engineering Expectations
Technical Design Template

It will now be added as a dependency of the extensions that use it, to avoid embedding it in all other clients that only need surface-level NetworkProtection functionality.
@samsymons samsymons requested a review from quanganhdo August 23, 2024 18:46
* main:
  Unified feedback form for Privacy Pro (#922)
  bump privacy-dashboard to 5.1.1 (#955)
  Bump Tests/BrowserServicesKitTests/Resources/privacy-reference-tests from `afb4f61` to `6133e7d` (#962)
  Freemium PIR - Add Desktop RMF Attribute (#960)
  Fix address bar queries when doing math expressions (#952)
@samsymons samsymons merged commit db0a7b4 into main Aug 28, 2024
7 checks passed
@samsymons samsymons deleted the sam/remove-wireguard-dependency branch August 28, 2024 03:04
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.

2 participants