Skip to content
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

Unexpected ID3 file Identifier, on HLS stream #2632

Closed
sanandrea opened this issue Mar 30, 2017 · 11 comments
Closed

Unexpected ID3 file Identifier, on HLS stream #2632

sanandrea opened this issue Mar 30, 2017 · 11 comments
Assignees

Comments

@sanandrea
Copy link

Hi,
I have a crash on an HLS stream found and the crash log is on this gist .

The same stream on the iOS AVPlayer does not have problems. I cannot post the link to the stream because credentials are needed to access the CDN of the stream. I have captured the chunk I thought has the wrong ID3 metadata. I have attached the chunk on which the streaming fails (u-6400-a-48-1-1.ts and the extracted aac source.aac)

thanks in advance

chunk.zip

@ojw28
Copy link
Contributor

ojw28 commented Mar 30, 2017

It looks like you're using ExoPlayer V1. You should move to ExoPlayer V2 for more robust ID3 handling.

@sanandrea
Copy link
Author

@ojw28 Thanks for your reply. I have also tried on the V2 ExoPlayer (some months before at commit 0cacd38 => Merge: dbebaa8 75eb047 of 05/01/2017) but I encountered the same problem. I think I have tried before the release V2.2.0. I'm gonna make a new check today and let you know.

@ojw28
Copy link
Contributor

ojw28 commented Mar 30, 2017

Cool. Note there are a couple of remaining fixes in flight, which are tracked by #2604. I think your case is different though, and hopefully already fixed. If not, please respond to that issue. Note we wont be able to assist without sample content (provided privately if necessary via dev.exoplayer@gmail.com).

@sanandrea
Copy link
Author

Hi @ojw28 I finally managed to have a working test stream. I have tried with the latest ExoPlayer (release-v2 @ 2.3.1) and still have a problem related to timedMetaData. I am attaching the log
sample_app_error_timedmetadata.log.zip

@ojw28 ojw28 reopened this Apr 10, 2017
@AquilesCanta AquilesCanta self-assigned this Apr 10, 2017
@AquilesCanta
Copy link
Contributor

I might be wrong, but I think the chunks are hosted in a different server than the playlist, and it's giving me a 403. Can you fix it?

@sanandrea
Copy link
Author

@AquilesCanta I am sorry, I realized later, that the content on the CDN is geoblocked. I have sent an email to dev.exoplayer@gmail.com with the instructions. thank you!

@ojw28
Copy link
Contributor

ojw28 commented Apr 12, 2017

I gave the stream a try. It seemed to be doing something really weird where the video tracks were periodically being identified as audio, which was breaking playback. I couldn't reproduce any ID3 related failures, however. Are you sure you're still seeing ID3 failures, and not other failures related to incorrect track identification (which I suspect is an issue with the streams, although I didn't have time to confirm).

@AquilesCanta
Copy link
Contributor

I haven't gotten any ID3 errors, either. Please try the dev branch and let me know if you are still getting the same ID3 decoding error. There has been some work on that area.

The provided chunk does not declare any video (even though the stream in which it is presented does contain video). ExoPlayer does not currently support structural changes in the TS, except for specific cases under which the removal of video does not fall. As a workaround, you could add a black screen video to the chunks that have no video. Adding support for dynamic changes in the stream tracks is not on the roadmap, unfortunately.

@sanandrea
Copy link
Author

Hi @AquilesCanta we have tried the following stream http://csm-e.cds1.yospace.com/csm/extlive/skydeprd01,263032-test01hls-audio.m3u8 it should be still working (via the CDN trick). News is that we have encountered the issue on the latest Exoplayer version 2.4.0 with the same exact error, but version 1.5.8 works perfectly and parses all timedMetadata.

@AquilesCanta
Copy link
Contributor

Everything was working smoothly until I started getting a 403, around after 2 minutes of playback. Does the issue reproduce immediately for you?

@kashif135
Copy link

Any ETA when this bug fix will be released please?

@google google locked and limited conversation to collaborators Aug 24, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants