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

Hls Flash video reproduced with uncorrect aspectratio if m3u8 is without RESOLUTION value #1790

Closed
ale-grosselle opened this issue Jul 25, 2016 · 6 comments

Comments

@ale-grosselle
Copy link
Contributor

Hi,
If you try to reproduce this m3u8:
http://thronqa-view.thron.com/api/xpublisher/resources/weebopublisher/getContentDescriptor.m3u8?clientId=thronqa&contentId=9f63b7df-6060-45ec-87b2-26407463ddb8&channelType=STREAMHTTPIOSHD&v=10

you will see the video with incorrect aspect ratio:
image

Proper playback should show this aspect ratio:
image

@ale-grosselle
Copy link
Contributor Author

ale-grosselle commented Jul 25, 2016

I have correct this behavior taking the dimension, if aren't set, from video dimension into _mediaTimeHandler method

Fix pull request: #1791

@rafa8626
Copy link
Contributor

rafa8626 commented Aug 2, 2016

@ale-grosselle Closing this since it is now merged on the master branch

@rafa8626 rafa8626 closed this as completed Aug 2, 2016
@rafa8626
Copy link
Contributor

@ale-groselle could you provide a URL with an HLS file to test this and make sure we will cover thos scenario for the 3.x-dev branch we are currently working, please?

@rafa8626
Copy link
Contributor

My apologies I thought the link you provided above didn't work anymore

@ale-grosselle
Copy link
Contributor Author

Hi,
I have tested this m3u8 url:
http://thronqa-view.thron.com/api/xpublisher/resources/weebopublisher/getContentDescriptor.m3u8?clientId=thronqa&contentId=9f63b7df-6060-45ec-87b2-26407463ddb8&channelType=STREAMHTTPIOSHD&v=10

and it works correctly; isn't it?

Il 11/ago/2016 21:03, "Rafael Miranda" notifications@github.com ha
scritto:

My apologies I thought the link you provided above didn't work anymore


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#1790 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF4xoAzzb4GeQ-xyY6AKQVctazQ9rUFzks5qe3ITgaJpZM4JT8B-
.

@rafa8626
Copy link
Contributor

Yes it does

@rafa8626 rafa8626 mentioned this issue Nov 7, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants