Skip to content

allow external app or adb shell to connect or disconnect vpn.#6

Open
yaoxinghuo wants to merge 28 commits intodlenski:masterfrom
yaoxinghuo:master
Open

allow external app or adb shell to connect or disconnect vpn.#6
yaoxinghuo wants to merge 28 commits intodlenski:masterfrom
yaoxinghuo:master

Conversation

@yaoxinghuo
Copy link

@yaoxinghuo yaoxinghuo commented Jun 10, 2019

Related Issue: #5
Related Issue: cernekee#17

Example to call from adb:

adb shell am start -a android.intent.action.MAIN -e app.openconnect.api.profileName profileName app.openconnect/.api.ConnectVPN
adb shell am start -a android.intent.action.MAIN app.openconnect/.api.DisconnectVPN

dlenski and others added 28 commits August 6, 2018 04:09
Need to use very recent versions of @dlenski's openconnect, incorporating
the multi-protocol API into the Java bindings
(https://github.com/dlenski/openconnect)
…system/bin/sh

Incredibly frustrating… why is this happening?
- Possibly because stdin is closed?
- Possibly because the output buffer isn't big enough?
Latest LibOpenConnect has two constructors, one of which accepts a User-Agent string.
- match whitespace convention elsewhere in project (TAB→4sp)
- remove debugging comments
- remove files that appear to do nothing (app/src/main/res/layout-v26/*)
@yaoxinghuo
Copy link
Author

@dlenski Hey,
I have made the pull request(not fully tested), you may test and check it, also you may have some tidy work because of my IDE env or my coding styles. Some codes from: https://github.com/schwabe/ics-openvpn -_-

@NgoHuy
Copy link

NgoHuy commented Jul 1, 2020

I think this pull request is not for privacy if some one can disconnect you from secure vpn.

@yaoxinghuo
Copy link
Author

You can control which app has this permission in settings.

@NgoHuy
Copy link

NgoHuy commented Jul 2, 2020

it's still exposing the risk malicious apps can disconnect you from secure vpn.

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