Sean-Walker0 commented on PR #7820: URL: https://github.com/apache/shenyu/pull/7820#issuecomment-6286156347
All four points addressed in the new commit: 1. **Leftover javadoc fragment** removed — `forwardWithCircuitBreaker` renders normally again. 2. **413 instead of 500**: the streaming branch now throws `ResponseStatusException(PAYLOAD_TOO_LARGE)`, which `GlobalErrorHandler` already honours — same client error as the known-length branch. 3. **Per-subscription counter**: the reset moved into `Flux.defer`, so every subscription restarts from zero instead of resuming the decorator field (kept your suggested shape rather than `doOnSubscribe` mutation). 4. **Test assertions**: `verifyErrorSatisfies` now asserts the exception type and the `PAYLOAD_TOO_LARGE` status. 25/25 module tests green, checkstyle clean. -- 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]
