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

Fix VPN IPv6 connectivity #1954

Merged
merged 3 commits into from
Dec 11, 2023
Merged

Fix VPN IPv6 connectivity #1954

merged 3 commits into from
Dec 11, 2023

Conversation

samsymons
Copy link
Collaborator

Task/Issue URL: https://app.asana.com/0/0/1206136376883995/f
Tech Design URL:
CC:

Description:

This PR pulls in an IPv6 fix.

Steps to test this PR:

  1. Test that NetP connects and the status UI doesn't show the port

Internal references:

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

Copy link
Contributor

@diegoreymendez diegoreymendez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works well!

samsymons added a commit to duckduckgo/BrowserServicesKit that referenced this pull request Dec 11, 2023
Required:

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

Description:

This PR fixes a regression related to IPv6.
@samsymons
Copy link
Collaborator Author

Test report seems to have gotten out of sync somehow, but I'll merge this PR as it's updating the macOS browser to a breaking BSK version. I ran these tests a few times and it looked like the Privacy Test Pages site was having issues, but it has settled down now.

@samsymons samsymons merged commit f530310 into main Dec 11, 2023
16 of 17 checks passed
@samsymons samsymons deleted the sam/fix-ipv6-connectivity branch December 11, 2023 21:34
samsymons added a commit that referenced this pull request Dec 11, 2023
Task/Issue URL: https://app.asana.com/0/0/1206136376883995/f
Tech Design URL:
CC:

Description:

This PR pulls in an IPv6 fix.
ayoy pushed a commit that referenced this pull request Dec 12, 2023
Task/Issue URL: https://app.asana.com/0/0/1206136370427438/f

Description:
This PR pulls in an IPv6 fix from #1954.

The BSK bump in this PR was the next tag after the version used in the 1.68.0 release, so no hotfix branch/tag was needed.
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