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
{{ message }}
Repository navigation
HTTP receiver should not expose bookkeeping functions #135
In receiver_binary.js there is a check() function which is used to validate the input. It is currently defined on BinaryHTTPReceiver.prototype but I don't think it should be exposed at all. It's the SDK's job to check input and validate it, not the user of the SDK, so why expose it? I suspect it's useful for testing. In fact, that's how the tests are written at the moment. The same exists for the structured receiver.
Since the parse() function already calls check(), I think it's reasonable to remove this function from the prototype and make it internal to the module.
Activity