Re: [I] [Bug] http connector in common protocol has a json deserialization error (eventmesh)

["qqeasonchen (via GitHub)" <gi...@apache.org>]

qqeasonchen commented on issue #5233:
URL: https://github.com/apache/eventmesh/issues/5233#issuecomment-5519453061

   **Resolution.** Closing via #5328.
   
   **Root cause.** The stack trace targets the *old* `openconnect` HTTP connector (`CommonProtocol.lambda$setHandler$0`), which called `JsonUtils.parseObject(jsonString, String.class)` on a JSON *object* body (`{"name":"Andy"}`) — Jackson can't deserialize an object into `String` and threw. That framework was removed by the uni-architecture redesign; the class no longer exists on `develop`.
   
   **What landed.** The rewritten HTTP source connector passes the raw request body through as event data (no brittle String re-parse), and #5328 hardened the replacement `ConnectorRuntime` so a deserialization/serialization failure on one event no longer kills the worker thread: it is logged + skipped + counted, and the rest of the batch still publishes (regression tests in `ConnectorRuntimeTest`).
   
   Since you ticked "willing to submit PR" — thank you! The fix direction ended up being framework removal + hardening rather than a one-line parse change, so no patch from your side is needed. If you can still reproduce a JSON error on `develop` after #5328, please reopen with the payload. Thanks for the report!
   


-- 
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: issues-unsubscribe@eventmesh.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@eventmesh.apache.org
For additional commands, e-mail: issues-help@eventmesh.apache.org