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

Listen for MDM config changes and rebuild TunnelService VPN #4040

Closed
jamilbk opened this issue Mar 7, 2024 · 0 comments · Fixed by #4181
Closed

Listen for MDM config changes and rebuild TunnelService VPN #4040

jamilbk opened this issue Mar 7, 2024 · 0 comments · Fixed by #4181
Assignees
Labels
area/android_client Issues related to the Android client business_value/high Required by > 50% of our customer base
Milestone

Comments

@jamilbk
Copy link
Member

jamilbk commented Mar 7, 2024

Need to listen for changes to the following fields in the AppRestrictions within TunnelService and then restart the Connlib session if they change:

token
allowedApplications
disallowedApplications

Unfortunately we can't call BuildVpnService instead because that generates a new fd which connlib won't know how to receive. So we should simply call session.disconnect and then Session.connect again.

@jamilbk jamilbk added the area/android_client Issues related to the Android client label Mar 7, 2024
@jamilbk jamilbk added this to the 1.0 GA milestone Mar 11, 2024
@jamilbk jamilbk added the business_value/high Required by > 50% of our customer base label Mar 11, 2024
github-merge-queue bot pushed a commit that referenced this issue Mar 22, 2024
Fixes #4040

Depends on #4180

---------

Signed-off-by: Jason Elie Bou Kheir <5115126+jasonboukheir@users.noreply.github.com>
Co-authored-by: Jamil <jamilbk@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/android_client Issues related to the Android client business_value/high Required by > 50% of our customer base
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants