-
Notifications
You must be signed in to change notification settings - Fork 36
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
Download error recently on Android for some podcasts #2161
Comments
@EricKerby thanks for creating this. Funny coincidence...I was just made aware of this in our Discord and have been investigating it for the past 10 minutes. It seems the error is happening because after updating the react-native-background-downloader library in v4.16.0, some episodes are now returning a "too many redirects" error message. I imagine there must be a way for us to increase the number of redirects allowed by the downloader service. For more context, one of the podcasts with the error is Pod Save America. Here is a link to the mp3 from their RSS feed: And I just checked New Heights with Jason and Travis Kelce, and their mp3 looks like: When I run curl on these it says 6 redirects. Anyway, hoping to find a quick solution for increasing the number of redirects allowed. |
A fix for this should be in v4.16.1. |
My fix was incomplete. I only fixed it for urls prefixed with the pdst.fm domain, but it turns out there are many other redirect tracking troublemaker domains... The fix I put in was to send pdst.fm mp3 urls to our "url resolving endpoint," then to send the final resolved url to the front-end. Ideally a more proper fix for this would happen on the client-side, but I have barely any Android skills, which makes it difficult for me to fix. Maybe however the code can be done with just JavaScript and I am overthinking it. I created kesha-antonov/react-native-background-downloader#15 for @kesha-antonov's library we use for downloading mp3, and it'd be nice if the library handled "too many redirects" itself, but based on the fact no one else has created an issue for it sooner, maybe it's not a common problem people are running into. Here are more examples of podcast urls that are failing due to "too many redirects":
|
not like podverse will migrate to the RN-fetch-blob family of downloaders, but these ones do work with redirect; |
@lovegaoshi thanks for your help. I didn't realize it was so easy to follow redirects with a HEAD request and axios (which we're using in PV). I started by trying to use a Anyway, I think I have a fix implemented for Podverse Android in v4.16.2. |
This should be fixed for good in v4.16.2 🤞 |
More download errors, this time for BBC podcasts 😩 When I check Logcat, it looks like they have an insecure URL somewhere in their redirects, and it results in a "cleartext not allowed" error. We have an API service that is supposed to get the HTTPS version of URLs, but it apparently isn't working for this case. |
New fix attempted in v4.16.3. |
I
The download is simply stucking. The direct playback is working. |
i cant reproduce with the same podcast. is this reproducible and could u post adb logs and/or willing to use android studio or a debug build to pin down the issue? |
In General yes. Connecting via adb is no issue I think. Is it enough to connect via USB and run For the debugging approach: it depends how much time I need for installing Android Studio (manjaro linux - I expect it is fast) and for debugging (I never did this). Regarding debug builds: I will not use binary packages from everyone. If they are build by the official maintainer ( @mitchdowney ) it would be ok. |
I'm recent weeks I started experiencing what seems like #1736 but on Android and for just some podcast feeds. I've tried logging out and back in on Podverse and unsubscribing and resubscribing to the affected podcasts with no success. Playing a stream works, but downloading an episode quickly fails and shows just "Error" in the My Library Downloads. Downloads of many other podcasts continue working fine.
Podcasts with issue for me are New Heights and The Bobby Bones Show.
Using Podverse 4.16.0 Build 5262 on Android 14 on a Pixel 7
The text was updated successfully, but these errors were encountered: