Add Tasker Actions for managing Downstreams#711
Add Tasker Actions for managing Downstreams#711XamarinDeveloper wants to merge 1 commit intoMygod:masterfrom
Conversation
Mygod
left a comment
There was a problem hiding this comment.
No Tasker functionality will be merged without proper permission checks. This app handles dangerous system networking stack and should not be exposed.
|
Doesn't the action I've added require root privileges? Edit: Also, if the permission check is necessary, how can I have a permission check parallel to a root check so if any one of them exists, it'll work? (Right now, my app denies permission check for the Tasker tether actions, although I have granted root) |
|
The action uses root, which is different from requiring the caller (presumably Tasker but could also be a malicious app) to have appropriate permissions. VPN Hotspot app caches and reuses root sessions, which also makes it hard to see when root is used but that's besides the point. |
|
Ah ok, alright, which permissions should I check for? this one doesn't have to do directly with tethering, is secure settings alone good? And would adding a checkbox to the settings where one could allow bypassing the permission checks (with a pop-up warning upon trying to enable it) be an acceptable compromise? (I mean with these kinds of apps that are made for power users I believe in the end the choice should be theirs) |
Needed it so implemented it.