mirror of
https://github.com/advplyr/audiobookshelf-app.git
synced 2026-07-25 14:08:35 +02:00
BUG: Playing downloaded file crashes android launcher (OnePlus) #49
Assignees
adam (Adam Melkus)
Clear assignees
adam (Adam Melkus)
No Assignees
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: starred/audiobookshelf-app#49
Reference in New Issue
Block a user
Originally created by @TheBinaryLoop on GitHub (Dec 31, 2021).
Steps to reproduce
Expected behaviour
Actual behaviour
Environment data
Audiobookshelf Version:
Android Issue
Android version: 11
Device model: OnePlus 8 Pro (IN2023)
Stock or customized system: Stock (OnePlus)
iOS Issue
iOS Version:
iPhone model:
@rubbo898 commented on GitHub (Dec 31, 2021):
Same here, my oneplus 8 crash and I have to reboot the whole device!
Both phone and app are updated to the last realese
@advplyr commented on GitHub (Dec 31, 2021):
Yeah this seems to be about 10% of users and because I can't reproduce the problem on any of my devices it is difficult to debug. For most people it seems if they choose a different folder to download to it can work that way.
That could be a temporary solution to try
@rubbo898 commented on GitHub (Dec 31, 2021):
I've just tried to switch folder but same results, phone crashed
@advplyr commented on GitHub (Dec 31, 2021):
I thought I had finally replicated this when I tested on a Nexus 6 playing the downloaded audiobook was causing the phone to lock up and flicker for a few seconds before crashing. It was also showing "System UI is not responding".
Then it started working normally, so I think that was just an emulator hardware issue.
Is there any information shown about the crash? Does it crash instantly when pressing play? What is happening that is forcing you to restart the device?
@rubbo898 commented on GitHub (Dec 31, 2021):
The device become unresponsive with display blinking and I'm only able to press the power button to reboot. Starting it stop at the recovery with logs of the crash but I can't understand how to collect and share them with you
@TheBinaryLoop commented on GitHub (Dec 31, 2021):
I'm going to try and use adb to collect some logs. I'll post them here if I find something
@TheBinaryLoop commented on GitHub (Dec 31, 2021):
I found these two lines, but i don't know what they mean:
12-31 22:53:23.391 1725 5235 E ScreenModeService: getAppToken ActivityRecord{9c75691 u0 com.audiobookshelf.app/.MainActivity t32416}
12-31 22:53:23.391 1725 5235 E ScreenModeService: setRefreshRate token ActivityRecord{9c75691 u0 com.audiobookshelf.app/.MainActivity t32416} rate 0 tmpRate 0
@advplyr I captured the complete crash via logcat but i cant seem to find any important information. If you want i can send you the full trace privately.
@advplyr commented on GitHub (Jan 1, 2022):
I'm not sure what that means either. It would be great if you can send it to me via email advplyr@protonmail.com
@advplyr commented on GitHub (Jan 1, 2022):
When looking through the logs the error I'm seeing has to do with reading the locally downloaded cover image.
If you can remove a cover on one of your audiobooks on the server and download it in the mobile app, that would help us see if it is just the cover images.
That log is very useful, thanks for sending that.
@TheBinaryLoop commented on GitHub (Jan 1, 2022):
Without the cover, it works like a charm.
@advplyr commented on GitHub (Jan 1, 2022):
That is great news, finally some progress on this issue.
Now to see if @rubbo898 has the same issue
@TheBinaryLoop commented on GitHub (Jan 1, 2022):
How hard is it to fix this issue?
@advplyr commented on GitHub (Jan 1, 2022):
Anywhere from "just needed to put a space there" to "we have to start from scratch to fix it".
@TheBinaryLoop commented on GitHub (Jan 1, 2022):
It looks like the path, that was build by the app is wrong:
content://com.android.externalstorage.documents/tree/primary%3AAudiobooks/document/primary%3AAudiobooks
instead of:
content://com.android.externalstorage.documents/tree/primary/Audiobooks/...
@advplyr commented on GitHub (Jan 1, 2022):
I don't think so. When the uri is decoded it becomes:
content://com.android.externalstorage.documents/tree/primary:Audiobooks/document/primary:Audiobooks
Although it might be wrong, I don't think there should be a / there
@TheBinaryLoop commented on GitHub (Jan 1, 2022):
I meant the primary:Audiobooks part... I don't think that part should be in there multiple times
@advplyr commented on GitHub (Jan 8, 2022):
The main thread for this issue will be #35