-
-
Notifications
You must be signed in to change notification settings - Fork 312
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
Asking for too much permissions #270
Comments
All these permissions are required by AndrOBD, since the permission handling of AndrOID did change a lot.
All permissions are listed, and commented in the manifest file within the project. |
Could you take a look in the sourcecode of https://github.com/libre-tube/LibreTube ? Its Android 5.0+ and do not require any of such deep permissions like AndrOBD do at the moment. |
After looking around some more, it would make sense to bump the minimum Android version to 5.0. https://punchthrough.com/android-ble-guide/ Quote: Note: As explained in our Android BLE Development Tips article, we assume the app targets a minimum of API 21 (Android 5.0) due to the availability of better BLE APIs such as BluetoothLeScanner and ScanFilter. We start this section by introducing the main classes from the Android SDK we’ll be using._ The lowest possible Android version F-Droid is running on is btw. Android 6.0 |
Describe the bug
Recent AndrOBD version ask for way too much permissions with no need.
It sort of ask for "give me access to everything". Pictures, music, full storage.
I also do not understand why it ask for Network access. I looked into the different options but was not able to find any functionality that is internet or network based. Is it for the MQTTPublisher addon?
I spend some minutes on a walk thinking what would be best example to show how the permissions should look like. I think i found the two best examples:
First and general:
Take a look into the permissions implementations of LibreTube. They work great for a full app with internet access that is downloading things from the internet into the internal memory of the phone.
Second:
Take a look into the permissions of OsmAnd. There are (huge) downloads are happening into the internal memory and also addons are available. But the app does not ask you for access to simply the whole memory of the phone like AndrOBD does at the moment.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
The minimal permissions that are required to my knowledge for a OBD-Bluetooth-App without internet access are:
"Nerby devices" (for Bluetooth)
"Sensors" (also somehow Bluetooth related to my understanding)
Some additional that are optional and could make sense for some usecases:
"Network" (maybe for MQTT?)
"Notifications" (maybe for some other things?)
Those should never be asked for:
Files, Music and audio, Photos and videos
Important details on your environment:
The text was updated successfully, but these errors were encountered: