{"action":"created","issue":{"url":"https://api.github.com/repos/youtube/js_mse_eme/issues/18","repository_url":"https://api.github.com/repos/youtube/js_mse_eme","labels_url":"https://api.github.com/repos/youtube/js_mse_eme/issues/18/labels{/name}","comments_url":"https://api.github.com/repos/youtube/js_mse_eme/issues/18/comments","events_url":"https://api.github.com/repos/youtube/js_mse_eme/issues/18/events","html_url":"https://github.com/youtube/js_mse_eme/issues/18","id":185190964,"number":18,"title":"Incorrently add BMFF for PlayReady tests and 500 license error","user":{"login":"calvaris","id":1021526,"avatar_url":"https://avatars.githubusercontent.com/u/1021526?v=3","gravatar_id":"","url":"https://api.github.com/users/calvaris","html_url":"https://github.com/calvaris","followers_url":"https://api.github.com/users/calvaris/followers","following_url":"https://api.github.com/users/calvaris/following{/other_user}","gists_url":"https://api.github.com/users/calvaris/gists{/gist_id}","starred_url":"https://api.github.com/users/calvaris/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/calvaris/subscriptions","organizations_url":"https://api.github.com/users/calvaris/orgs","repos_url":"https://api.github.com/users/calvaris/repos","events_url":"https://api.github.com/users/calvaris/events{/privacy}","received_events_url":"https://api.github.com/users/calvaris/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":25,"created_at":"2016-10-25T18:03:47Z","updated_at":"2016-11-18T07:52:04Z","closed_at":null,"body":"Answering https://github.com/youtube/js_mse_eme/pull/17#issuecomment-255856808 :\n\n> The initData should have the keys as expected for clearKey regardless if we need them or not. This points to a problem with the initData the UA is providing. I suspect that you are using a different EME version other than 0.1b. That would explain why your initData is not the expected initData.\n\nThe story begins when I am fixing a problem with PlayReadyH264Video EME test as we were selecting the wrong decryptor. This is irrelevant for now as when I encountered this problem I stashed that code and just forced the selection of the PlayReady decryptor when I hit the JS issue and later the 500. Let's forget about this now then.\n\nI am using v0.1b here, I am not building anything else.\n\nHow is the initData going to have the keys expected for ClearKey when we are providing PlayReady initData? FTR: The initData, the are providing for the PR engine, the generated key request and the server answer can be found at http://pastebin.com/neLhSDr4 . Of course this is after applying the fix I provided at https://github.com/youtube/js_mse_eme/pull/17, otherwise I would get no generateKeyRequest call.\n\n> I also was not able to reproduce the 500s on the license server. Can you provide more information with how you are running the tests or if you have made any modifications.\n\nYou have the info about the 500 on the pastebin link above.\n\nI am running the following link for the tests http://mylocalipaddress/mstest/2016.html?enablewebm=false&command=run&test_type=encryptedmedia-test&tests=16 which contains changes at http://pastebin.com/w5tSPvsm . As you can see it is only the fix and some useful debugging (part of it was submitted, some was rejected and some accepted and reverted later, but I still find it useful).\n\n> I would be happy to look into this further but I need some more information on the failure. A complete stacktrace would be helpful, information on the UA EME version, and information about the 500 on the license server.\n\nInfo about the 500, you have it already. For UA EME version we run WebKitForWayland on a Raspberry Pi 2. I don't know what you mean by stacktrace? JS stacktrace? It is difficult to get one cause of having an embedded device but I can tell you that the get onneedkey event handler, then generateKeyRequest is called, that triggers onkeymessage handler that calls licenseManager.acquireLicense that ends up calling licenseManager.requestLicense that answers that I told you and that ends up calling the callback that calls video.addKey with the 500 answer.\n\n> At the moment I don't have an immediate issue with the fix you are proposing but I think it is a red herring for the problem you are experiencing. I'm closing this request as a result. Please feel free to file a bug report so we can analyze this problem further.\n\nI opened this issue then. Let's see if it is a red herring or not by analyzing the problem further :)\n"},"comment":{"url":"https://api.github.com/repos/youtube/js_mse_eme/issues/comments/261469744","html_url":"https://github.com/youtube/js_mse_eme/issues/18#issuecomment-261469744","issue_url":"https://api.github.com/repos/youtube/js_mse_eme/issues/18","id":261469744,"user":{"login":"calvaris","id":1021526,"avatar_url":"https://avatars.githubusercontent.com/u/1021526?v=3","gravatar_id":"","url":"https://api.github.com/users/calvaris","html_url":"https://github.com/calvaris","followers_url":"https://api.github.com/users/calvaris/followers","following_url":"https://api.github.com/users/calvaris/following{/other_user}","gists_url":"https://api.github.com/users/calvaris/gists{/gist_id}","starred_url":"https://api.github.com/users/calvaris/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/calvaris/subscriptions","organizations_url":"https://api.github.com/users/calvaris/orgs","repos_url":"https://api.github.com/users/calvaris/repos","events_url":"https://api.github.com/users/calvaris/events{/privacy}","received_events_url":"https://api.github.com/users/calvaris/received_events","type":"User","site_admin":false},"created_at":"2016-11-18T07:52:04Z","updated_at":"2016-11-18T07:52:04Z","body":"You were quick :) I deleted de message cause I realized that I was printing it as a regular string and as it contains null characters the string was truncated during printing."}}
{"actor":{"display_login":"calvaris"}}