Skip to content

allow users to select/deselect network resources - #86

Merged
mlsmaycon merged 46 commits into
mainfrom
feature/allow-selecting-routes
Nov 21, 2025
Merged

mlsmaycon merged 46 commits into
mainfrom
feature/allow-selecting-routes

Conversation

@doromaraujo

@doromaraujo doromaraujo commented Sep 22, 2025 •

Copy link
Copy Markdown
Contributor

This PR adds toggle switches on the resources/networks tab to allow users to select/deselect network resources.

When either a network change is detected from the server or a resource is selected/deselected, a new tun file descriptor is obtained from the service side and sent to the engine to be renewed internally with the new route definitions.

The engine is also notified when the user toggles a network resource accordingly.

The marshalled routes are being sent from go client separated
by semicolons when provided in the IFace configureInterface
callback, but separated by commas on onNetworkChanged call
Now VPNService adds an OnRouteChanged listener
to NetworkChangeNotifier, which queues TUN renewal via a
looper thread
Also remove usage of ServiceAccessor from NetworksFragment
Adapter calls a switch toggle handler defined in
NetworksFragment. Fragment, through its view model, reaches
all the way to the engine to call the go client functions.
with read/write access on android's context
@doromaraujo doromaraujo changed the title allow selecting routes allow users to select/deselect network resources Oct 8, 2025
@doromaraujo
doromaraujo marked this pull request as ready for review October 8, 2025 15:51
To group android client's file paths parameters, such as
configurationFilePath and stateFilePath
@pappz

pappz commented Oct 17, 2025

Copy link
Copy Markdown
Collaborator

The pop-up message "Your network administrator changed a configuration..." still appears when I add or remove a route on the Dashboard.

@mlsmaycon
mlsmaycon merged commit 79cf6e6 into main Nov 21, 2025
6 checks passed
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.

3 participants