Skip to content

Conversation

@MarkusPettersson98
Copy link
Contributor

@MarkusPettersson98 MarkusPettersson98 commented Nov 11, 2025

This PR moves the WireGuard port selection into the obfuscation settings data structure. Note that the port selection is now scoped to the new anti-censorship method named WireGuard port, and is no longer shared with LWO.

Why is this branch named remove-openvpn-part5?

This PR seeks to split up #9292 even more, since that PR both moves the WireGuard port to obfuscation menu AND changes the management interface to treat port as an obufscation method. This PR plucks out just the changes for making the management interface to treat port as an obufscation method.


This change is Reviewable

@MarkusPettersson98 MarkusPettersson98 force-pushed the remove-openvpn-part4 branch 2 times, most recently from 02065db to 1542a3e Compare November 11, 2025 10:07
Serock3
Serock3 previously approved these changes Nov 11, 2025
Copy link
Contributor

@Serock3 Serock3 left a comment

Choose a reason for hiding this comment

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

:lgtm:

@Serock3 reviewed 21 of 58 files at r1, all commit messages.
Reviewable status: 21 of 58 files reviewed, all discussions resolved

Base automatically changed from remove-openvpn-part4 to main November 11, 2025 10:32
@MarkusPettersson98 MarkusPettersson98 dismissed Serock3’s stale review November 11, 2025 10:32

The base branch was changed.

@MarkusPettersson98
Copy link
Contributor Author

This branch is now based on main as of fc27754, which means that we "only" have to maintain this portion of the original #9227 PR. 🎉

@MarkusPettersson98 MarkusPettersson98 changed the title Remove OpenVPN - Part 3b Promote WireGuard Port to a proper obfuscation method Nov 11, 2025
@MarkusPettersson98 MarkusPettersson98 force-pushed the remove-openvpn-part5 branch 2 times, most recently from e8282b4 to 66916f8 Compare November 17, 2025 12:35
@MarkusPettersson98
Copy link
Contributor Author

Rebased on top of d1f1b8d. Hopefully this branch will see less breakage since OpenVPN is basically ripped out of the daemon at this point.

Copy link
Contributor

@Serock3 Serock3 left a comment

Choose a reason for hiding this comment

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

@Serock3 reviewed 2 of 58 files at r1, 4 of 9 files at r2, 13 of 16 files at r3, all commit messages.
Reviewable status: 24 of 59 files reviewed, all discussions resolved

@MarkusPettersson98
Copy link
Contributor Author

The plan is to let Android implement their client-side changes as part of this PR before merging.

Copy link
Contributor

@Serock3 Serock3 left a comment

Choose a reason for hiding this comment

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

@Serock3 reviewed 1 of 58 files at r1, 3 of 10 files at r4, all commit messages.
Reviewable status: 25 of 60 files reviewed, all discussions resolved

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.

5 participants