-
Notifications
You must be signed in to change notification settings - Fork 76
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
[IOpenCDM] Update with JSON-RPC interface #413
Conversation
@pwielders what do you think about making On one hand, the CDMs list is finite and sending an enum would be more efficient, but on the other, OpenCDM don't need to know what DRM types exist |
@VeithMetro on you question. Agree and do have a preference for enums, however the interface specification was initially once drafted by the Frauenhofer Institute and a s long that there was no severe technical reason to deviate from what they specified, we did not modify it. So for me, as it is is fine but do agree that for a communication overhead the enum would be much better. So for sake of backwards compatibility with what Frauenhofer once specified, I vote for keeping it a string.. |
After merging this PR, we have to also merge this PR with changes to the plugin