search

asterisk / node-ari-client

Node.js client for ARI. This library is best effort with limited support.
Other
250 stars 98 forks source link

Handle binary response type #63

Closed mpotra closed 7 years ago

mpotra commented 7 years ago

Fixes #62

chadxz commented 7 years ago

Thanks for submitting a fix! The change you made looks like the right thing, but it could be simpler. If you could fix that up we'll merge and cut a new release.

mpotra commented 7 years ago

You're right, it is much simpler to just add the buffer copy in the main file, and not do any changes to knownTypes. Submitted it as a new PR, replacement for #63

chadxz commented 7 years ago

closing in favor of #64 .