-
-
Notifications
You must be signed in to change notification settings - Fork 5
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
Fastlane has wrong location #2
Comments
OK, seems to have crossed: Fastlane location is now correct. Thanks also for the screenshots, which help clarify some of the permissions as well β and I rather wonder how you disable mobile data (no Looking forward to the other adjustment then β and thanks a lot! |
@IzzySoft thanks a lot for finding my repo. Always amazed at how fast you find my apps :D As for permission questions:
|
Big brother is waβ¦ err, my crawlers are reporting nicely to me π
Cool, thank YOU!
Does that mean the app requires root? I thought you were ustilizing Shizuku to not require root? Just asking, because if root is required we should mark that, to avoid disappointment. OK, let's see what I've set up here (must have guessed by the description already as all the perms are already on the green list): android.permission.QUERY_ALL_PACKAGES: needed to select apps to doze
android.permission.ACCESS_NETWORK_STATE: used to detect hotspot state to not disable wifi/data if hostpot is enabled
android.permission.READ_PHONE_STATE: to check if mobile data need to be disabled on Doze if configured
android.permission.ACCESS_WIFI_STATE: to check if WiFi needs to be disabled on Doze if configured
android.permission.CHANGE_WIFI_STATE: to disable WiFi on Doze if configured Different wording (only thing I've changed right now is |
It has a non root mode as explained in the readme. But yes if you want the full potential you need root.
All seems good to me. Do you want me to put that in the Readme too? |
OK, then we got that right. It does not require root (to work at all), but it can do more if it has root.
Not a bad idea I'd say β so folks know what to expect, and not disappointed when having no root access that exactly the thing they wanted is not there.
Can't hurt, but up to you of course β it's your repo here π |
You've set up
/fastlane/en-US
, which F-Droid won't find (but is no problem for IzzyOnDroid, where your app will show up with the next sync around 6 pm UTC β be welcome to pick a badge and link there e.g. from your Readme π). Correct locations for F-Droid would be/metadata/en-US
or, the standard location by fastlane definition,/fastlane/metadata/android/en-US
. While on it, please also add an icon (currently your app would show up without one β I added one manually here for IzzyOnDroid) and, if possible, a (few) screenshot(s).While I'm here, there are a few findings by the IzzyOnDroid scanners which F-Droid missed (as they do not have them in place):
This can easily be avoided with a minor addition to your
build.gradle
:For some background: that BLOB is supposed to be just a binary representation of your app's dependency tree. But as it's encrypted with a public key belonging to Google, only Google can read it β and nobody else can even verify what it really contains. More details can be found e.g. here: Ramping up security: additional APK checks are in place with the IzzyOnDroid repo.
I also wonder why EnforceDoze requests
android.permission.READ_PHONE_STATE
as well asandroid.permission.ACCESS_WIFI_STATE
,android.permission.CHANGE_WIFI_STATE
andandroid.permission.ACCESS_NETWORK_STATE
. Could you please clarify, so I can add that to your app's "green list" with the proper explanation? Thanks a lot!The text was updated successfully, but these errors were encountered: