atharvalade opened a new issue, #3128:
URL: https://github.com/apache/iggy/issues/3128

   ## Problem
   `deserializeVoidResponse` was changed from:
   - `status === 0 && data.length === 0`
   to:
   - `status === 0`
   
   This applies to all void commands, not just `SendMessages`.
   
   ## Why this is critical
   This removes a useful protocol safety check.
   Malformed or unexpected payloads for void commands are now silently accepted.
   
   ## Where
   - `foreign/node/src/client/client.utils.ts`
   
   ## Expected
   Keep strict empty-payload validation for true void commands.
   Handle `SendMessages` with a dedicated response deserializer instead of 
weakening global void validation.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to