Skip to content

VPN flags are now opt-in#3227

Merged
Mygod merged 1 commit into
masterfrom
novflag-port
Jan 20, 2026
Merged

VPN flags are now opt-in#3227
Mygod merged 1 commit into
masterfrom
novflag-port

Conversation

@Mygod
Copy link
Copy Markdown
Contributor

@Mygod Mygod commented Jan 19, 2026

This re-implements #1742. This allows plugin developers to use the default network directly rather than forcing them to use protect_path. They can still opt in to use protect_path to make sure that the correct underlying network is used but given the complexity (no plugins are implemented in JVM) and the small marginal benefits, I don't expect most to do this.

@Mygod Mygod requested a review from Copilot January 19, 2026 23:51
@Mygod Mygod merged commit 1dc60cb into master Jan 20, 2026
8 checks passed
@Mygod Mygod deleted the novflag-port branch January 20, 2026 04:02
@Mygod Mygod review requested due to automatic review settings March 23, 2026 22:06
cocomine added a commit to cocomine/shadowsocks-android that referenced this pull request Apr 19, 2026
* Update dependencies

* Fix profileName fallback missing

* Update dependencies

* VPN flags are now opt-in for plugins (shadowsocks#3227)

* Fix build staleness

* Bump version to 5.3.5-nightly

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* Add E2E test GitHub Actions workflow

Parameterize test-e2e.sh with env-var defaults so it works both locally
(macOS/ARM64) and on CI (Linux/x86_64). Add TARGET_ABI Gradle property
to build a single-architecture APK, and create an e2e-test workflow that
runs the VPN connectivity tests on an x86_64 emulator with KVM.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Mygod <contact-git@mygod.be>
Co-authored-by: Max Lv <max.c.lv@gmail.com>
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: cocomine <22698880+cocomine@users.noreply.github.com>
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.

1 participant