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

Add JSON support in communication with zencoder.com #5

Open
aleksey-korolev opened this issue Sep 10, 2012 · 3 comments
Open

Add JSON support in communication with zencoder.com #5

aleksey-korolev opened this issue Sep 10, 2012 · 3 comments

Comments

@aleksey-korolev
Copy link
Contributor

zencoder.com supports communication in both json (by default) and xml.
ZencoderClient.createJob(ZencoderJob job) returns Document and there's no way to instruct ZencoderClient to use JSON.
JSON support has to be added.
There are 2 options:

  1. ZencoderClient uses json format for communication and returns json response from server as a String which is later processed by developer's code upon developer's discretion.
  2. ZencoderClient parses json string from server and constructs JSON object using one of JSON libraries, e.g. jackson or json-simple.
@drallgood
Copy link
Contributor

hmmm...

Is there a reason why someone would need JSON and can't work with XML?

I didn't want to complicate the code by supporting two different output formats. That's why I never included JSON support.

@aleksey-korolev
Copy link
Contributor Author

Well, I think JSON would be less expensive in parsing, but this is just my speculation, I didn't do any tests yet.

@drallgood
Copy link
Contributor

Hmm.. Don't think so. Might depend on the parser, though.

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