-
-
Notifications
You must be signed in to change notification settings - Fork 19
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
initial fastlane structure #15
Conversation
Thanks! If you need a badge to link to your app in my repo, be welcome to take your pick 😃 |
Thanks! I added to the repo readme. And I'll improve the fastlane docs even more later on this week, really thanks for your pr! I've only a doubt about the permission section, I didn't find any info in your repo about: how do you define the permissions required by an app? Screenshot from apt.izzysoft.de/fdroid/index/apk/io.alexrintt.kanade. Those versions (v0.2.1 and v0.3.0) it has only So maybe it was queried from the past versions? (v0.1.0) when these permissions were still required? |
Cool, thanks!
I do not define permissions. Fdroid reads those from the APKs (Manifest IIRC). To compare, here's the output of package: name='io.alexrintt.kanade' versionCode='3' versionName='0.3.0' platformBuildVersionName='12' platformBuildVersionCode='31' compileSdkVersion='31' compileSdkVersionCodename='12' sdkVersion:'19' targetSdkVersion:'30'
uses-permission: name='android.permission.QUERY_ALL_PACKAGES'
uses-permission: name='android.permission.READ_EXTERNAL_STORAGE'
uses-permission: name='android.permission.WRITE_EXTERNAL_STORAGE'
… Obviously some dependency drags in the other two (yes, I saw your original Manifest only has the first). If you'd ask me to guess, I'd pick <manifest xmlns:android="http://schemas.android.com/apk/res/android" xmlns:tools="http://schemas.android.com/tools" package="io.alexrintt.sharedstorage">
<uses-permission android:name="android.permission.READ_EXTERNAL_STORAGE" />
</manifest> I have another good guess whom you could ask on that plugin 🙈 But that still doesn't tell us where <manifest xmlns:android="http://schemas.android.com/apk/res/android" package="io.alexrintt.sharedstorage.example">
<uses-permission android:name="android.permission.WRITE_EXTERNAL_STORAGE" />
<uses-permission android:name="android.permission.READ_EXTERNAL_STORAGE" />
<application android:icon="@mipmap/ic_launcher" android:label="Shared Storage Example"> 🤷♂️ |
I'm really confuse after debugging these apks. As you said it is probably coming from the third-party dependencies, so I removed all permissions from the shared storage example and the shared storage plugin itself, and then I compiled again... and it still prints WRITE and EXTERNAL permissions storage! lol. I'm using apktool to decompile the apk <!-- Final apk AndroidManifest.xml -->
<uses-permission android:name="android.permission.READ_EXTERNAL_STORAGE"/>
<uses-permission android:name="android.permission.WRITE_EXTERNAL_STORAGE"/> I even tried to find some subplugin/subpackages in the dependency tree that uses these permissions but I didn't find any (because the only dependency is shared storage itself and another dart based package which doesn't even has native android code). I'll investigate further tomorrow, find some answer, cause SAF is not intended to be something I would require permissions (at least in my head from what I understood, but probably I understood wrong). But thanks for your explanation, it helped a lot! |
That's really strange. Good luck then finding the culprit! |
Ok. I found: https://github.com/anggrayudi/SimpleStorage/blob/master/storage/src/main/AndroidManifest.xml. It's a native Android library used by shared_storage plugin. I completely forget about it, lol. Thanks man. |
Glad the riddle solved! |
as discussed in #13 – and here are some hints to it:
full_description.txt
should you need to update the description, or want to provide one for other languages.Enjoy!