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

Implement encryption support for HLS #69

Closed
andudo opened this issue Oct 3, 2014 · 3 comments
Closed

Implement encryption support for HLS #69

andudo opened this issue Oct 3, 2014 · 3 comments
Assignees

Comments

@andudo
Copy link
Contributor

andudo commented Oct 3, 2014

METHOD=AES-128

@perchrh
Copy link

perchrh commented Oct 22, 2014

In case this could be helpful to others, I wanted to mention that I've ported the support for encrypted streams from the Dailymotion fork of ExoPlayer to the dev-hls branch from Google.
The branch is available at https://github.com/perchrh/ExoPlayer/tree/dev-hls-google-crypto-support

This is the encrypted stream I've used for testing: http://playertest.longtailvideo.com/adaptive/oceans_aes/oceans_aes.m3u8

@andudo
Copy link
Contributor Author

andudo commented Oct 31, 2014

@perchrh Thank you, this was helpful. We'll push our own changes very soon.
BTW, In your stream I noticed that only the first chunk (TS file) has PMT in it, and all the other chunks have no PMT after PAT. This seems broken to me. Where did you get it?

@perchrh
Copy link

perchrh commented Nov 1, 2014

@andudo I found the stream in this list of test streams, http://www.jwplayer.com/html5/hls/
I'm not that into the TS format, so I cannot comment on the PMT/PAT.
Quicktime plays the video too without any noticeable problems. There are, perhaps, some common encoding mistakes that players tend to work around.

Let me know if you need more encrypted test streams!

Edit: I btw noticed that my normalizeIV() method has a trivial bug, it looks for leading '0x', but it should, according to the spec, also handle '0X' (capital x) in the IV. Fixed in the mentioned branch.

andudo added a commit that referenced this issue Nov 4, 2014
@andudo andudo closed this as completed Nov 7, 2014
@ojw28 ojw28 mentioned this issue Nov 20, 2014
@google google locked and limited conversation to collaborators Jun 28, 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

2 participants