CryoThrust commented on issue #12066: URL: https://github.com/apache/seatunnel/issues/12066#issuecomment-5541191063
The proposed contract looks appropriately narrow: keep the HTTP status and field unchanged, return a single client-facing exception message, and continue logging the full stack trace server-side.\n\nFor implementation, I would suggest a focused filter-level test that exercises both the (400) and generic exception (500) paths, asserts the JSON message contains the exception message without a newline or frames, and verifies the existing server-side error logging remains unchanged. It would also be useful to confirm whether is preferred over the root-cause message when the top-level exception adds request context. -- 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]
