You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I am using method payloads (and responses) that are not in JSON form, but base64 strings (strictly speaking still valid json) as my connected devices use cellular data, so I want to keep data usage to an absolute minimum. However, the MethodCall.Payload in this library is a map[string]interface{}. I forked your library to change it into just an interface{}.
Are you interested in a pull request? (and if so, would you prefer a new method (to keep it backwards compatible) or a change to MethodCall and MethodResponse?)
The text was updated successfully, but these errors were encountered:
IIRC when I was writing the code IoTHub accepted json only and now as understand this correctly it can be anything []byte, plus the library itself doesn't guarantee API changes yet. So I think it's worth changing it, but I'd prefer that RegisterMethod method signature remain the same and I'd like it to call the new one internally (say HandleMethod):
Thank you for creating this library :)
I am using method payloads (and responses) that are not in JSON form, but base64 strings (strictly speaking still valid json) as my connected devices use cellular data, so I want to keep data usage to an absolute minimum. However, the MethodCall.Payload in this library is a
map[string]interface{}
. I forked your library to change it into just aninterface{}
.Are you interested in a pull request? (and if so, would you prefer a new method (to keep it backwards compatible) or a change to
MethodCall
andMethodResponse
?)The text was updated successfully, but these errors were encountered: