kivy / kivy/python-for-android
String argument from Buildozer not decoded.
- Langage dominant
- Python
- Étoiles
- 8.9k
- Forks
- 2k
- Merge moyen
- 12 h 20 min
- PR mergées (30 j)
- 9
Description
### Checklist
- [ x] the issue is indeed a bug and not a support request
- [ x] issue doesn't already exist: https://github.com/kivy/python-for-android/issues
- [x ] I have a short, runnable example that reproduces the issue
- [ x] I reproduced the problem with the latest development version (`p4a.branch = develop`)
- [ x] I used the grave accent (aka backticks) to format code or logs when appropriated
### Versions
- Python: 3.11
- OS: WSL/Ubuntu
- Kivy: 2.2.1
- Cython: 0.29.33
- OpenJDK: 17
### Description
In buildozer.spec , setting:
```
android.extra_manifest_application_arguments = somefile.xml
```
Where `somefile.xml` contains:
```
android:usesCleartextTraffic="true"
```
results in an excaped string argument from Buildozer to p4a:
```
# Run ['/usr/bin/python3', '-m', 'pythonforandroid.toolchain', 'apk', '--bootstrap', 'sdl2', '--dist_name', 'oauth_gspread_example', '--name', 'OG', '--version', '0.1', '--package', 'org.reallyatest.oauth_gspread_example', '--minsdk', '21', '--ndk-api', '21', '--private', '/home/bobf/ex/oauth_gspread_example/.buildozer/android/app', '--permission', 'INTERNET', '--android-entrypoint', 'org.kivy.android.PythonActivity', '--android-apptheme', '@android:style/Theme.NoTitleBar', '--orientation', 'portrait', '--window', '--enable-androidx', '--copy-libs', '--add-source', '/home/bobf/ex/oauth_gspread_example/java', '--extra-manifest-application-arguments="android:usesCleartextTraffic=\\"true\\" "', '--arch', 'arm64-v8a', '--color=always', '--storage-dir=/home/bobf/ex/oauth_gspread_example/.buildozer/android/platform/build-arm64-v8a', '--ndk-api=21', '--ignore-setup-py', '--debug']
```
the key part to note is the value from `somefile.xml` has been quoted and escaped:
```
'--extra-manifest-application-arguments="android:usesCleartextTraffic=\\"true\\" "'
```
The value is written without interpretation to `AndroidManifest.xml` as an escaped string
```
" or "/>".
```
The last but one line in the `AndroidManifest.xml` snippet above should the same a the contents of `somefile.xml`
```
android:usesCleartextTraffic="true"
```
Presumably, Buildozer has to quote and escape in order to pass the argument, so the issue is assumed to be a failure of the p4a parser to decode the argument.
Since we presumably know Buildozer's (quote and escape) encoding algorithm, it should be possible to successfully decode. A simpler approach might be a new p4a argument receiving the file name, and Buildozer supplying the file name.
I didn't test but this may also apply to the `buildozer.spec` `android.extra_manifest_xml` option.
Guide de contribution
Ouvrir le guide de contribution
Piste de recherche
Commencez par suivre le traitement de --extra-manifest-application-arguments par p4a, depuis la commande indiquée dans l’issue jusqu’à la génération de AndroidManifest.xml ; comparez-le aux entrées de buildozer.spec et somefile.xml. Reproduisez l’exemple avec Python 3.11 sous WSL/Ubuntu, puis vérifiez que le manifest généré contient android:usesCleartextTraffic="true" en tant qu’attribut et que Gradle l’accepte.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- android, python
- Domaine
- build-system, mobile
- Type d'issue
- Bug
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 45/100