samuelclay / NewsBlur

NewsBlur is a personal news reader that brings people together to talk about the world. A new sound of an old instrument.
http://www.newsblur.com
MIT License
6.9k stars 1k forks source link

Android: App crashes on Export OPML #1891

Closed shuvashish76 closed 1 month ago

shuvashish76 commented 1 month ago

Logs:

FATAL EXCEPTION: main
Process: com.newsblur, PID: 5981
java.lang.SecurityException: No permission to write to /storage/emulated/0/Download/NewsBlur-OPML-newsblur76.xml: Neither user 10368 nor current process has android.permission.WRITE_EXTERNAL_STORAGE.
    at android.os.Parcel.createException(Parcel.java:1950)
    at android.os.Parcel.readException(Parcel.java:1918)
    at android.database.DatabaseUtils.readExceptionFromParcel(DatabaseUtils.java:183)
    at android.database.DatabaseUtils.readExceptionFromParcel(DatabaseUtils.java:135)
    at android.content.ContentProviderProxy.insert(ContentProviderNative.java:476)
    at android.content.ContentResolver.insert(ContentResolver.java:1588)
    at android.app.DownloadManager.enqueue(DownloadManager.java:1032)
    at i1.q.onClick(SourceFile:147)
    at android.view.View.performClick(View.java:6597)
    at android.view.View.performClickInternal(View.java:6574)
    at android.view.View.access$3100(View.java:778)
    at android.view.View$PerformClick.run(View.java:25906)
    at android.os.Handler.handleCallback(Handler.java:873)
    at android.os.Handler.dispatchMessage(Handler.java:99)
    at android.os.Looper.loop(Looper.java:193)
    at android.app.ActivityThread.main(ActivityThread.java:6718)
    at java.lang.reflect.Method.invoke(Native Method)
    at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:491)
    at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:858)

Instead of crash it should ask for the relevant permission to the user.

App version : 13.3.0 Source : F-Droid

sictiru commented 1 month ago

@shuvashish76 What device and android version is this from?

shuvashish76 commented 1 month ago

Lenovo Tab4 8 plus (TB-8704X), Android 9

sictiru commented 1 month ago

Pushed a fix as 13.3.1 to open beta testers.

sictiru commented 1 month ago

Pushed to production

shuvashish76 commented 1 month ago

Don't forget to push tags on releases for F-Droid...🫣