nats-io / nats.ts

TypeScript Node.js client for NATS, the cloud native messaging system
https://www.nats.io
Apache License 2.0
178 stars 13 forks source link

FIX #50 - Invalid JSON has misleading message (even if msg should be ignored) #51

Closed aricart closed 5 years ago

aricart commented 5 years ago

When receiving invalid JSON payloads, the client was presented with a partial message that associated the subject of the subscription rather than the collected (and potentially invalid message). Changed so that the invalid message is included - the invalid message will have the correct subject the parsed from the protocol line, etc).

Also updated tests showing that error must be checked.

coveralls commented 5 years ago

Pull Request Test Coverage Report for Build 164


Totals Coverage Status
Change from base Build 161: 0.3%
Covered Lines: 810
Relevant Lines: 916

💛 - Coveralls