🐛 Allow callFollowingErrorInterceptor when rejecting in ErrorInterceptorHandler#2543
Conversation
RequestInterceptorHandler and ResponseInterceptorHandler both support the callFollowingErrorInterceptor flag in their reject() methods, but ErrorInterceptorHandler.reject() was missing it, always emitting InterceptorResultType.reject and causing subsequent error interceptors to be skipped unconditionally. The errorInterceptorWrapper in dio_mixin.dart already handles rejectCallFollowing correctly, so only the handler method needed fixing. Fixes #2462 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Add a regression test to ensure calling ErrorInterceptorHandler.reject(error, true) inside QueuedInterceptorsWrapper continues to following error interceptors. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Code Coverage Report: Only Changed Files listed
Minimum allowed coverage is |
PR Review: Allow
|
Fixes #2462
RequestInterceptorHandlerandResponseInterceptorHandlerboth expose acallFollowingErrorInterceptorflag in theirreject()methods, butErrorInterceptorHandler.reject()was missing it — it always emitsInterceptorResultType.reject, unconditionally skipping any subsequent error interceptors.The
errorInterceptorWrapperindio_mixin.dartalready handlesrejectCallFollowingcorrectly, so only the handler method needed updating.