_retryCount is shared between multiple requests making it dangerous to infinite requests

[itsUndefined]

Describe the bug
Because _retryCount is not local var to _attemptRequest, it's possible to create infinite loops since a new request restarts the _retryCount to 0 for all pending requests.

To Reproduce

1. Add a shouldAttemptRetryOnResponse that starts a new request with the same intercepted client that succeeds ( e.g. a refresh )
2. Make an api request that fails all the time and calls shouldAttemptRetryOnResponse
3. Observe infinite looping

Expected behavior
I expected that after maxRetryAttempts attempts of the original request, the request would be cancelled

Please complete the following information):

• Flutter version: 3.35.4
• http_interceptor: 2.0.0

[itsUndefined]

I fixed the issue locally by changing the method _attemptRequest to the following:

Future<BaseResponse> _attemptRequest(BaseRequest request,
      {bool isStream = false, int retryCount = 0}) async {
    BaseResponse response;
    try {
      // Intercept request
      final interceptedRequest = await _interceptRequest(request);

      var stream = requestTimeout == null
          ? await _inner.send(interceptedRequest)
          : await _inner
              .send(interceptedRequest)
              .timeout(requestTimeout!, onTimeout: onRequestTimeout);

      response = isStream ? stream : await Response.fromStream(stream);

      if (retryPolicy != null &&
          retryPolicy!.maxRetryAttempts > retryCount &&
          await retryPolicy!.shouldAttemptRetryOnResponse(response)) {
        await Future.delayed(retryPolicy!
            .delayRetryAttemptOnResponse(retryAttempt: retryCount + 1));
        return _attemptRequest(request, retryCount: retryCount + 1);
      }
    } on Exception catch (error) {
      if (retryPolicy != null &&
          retryPolicy!.maxRetryAttempts > retryCount &&
          await retryPolicy!.shouldAttemptRetryOnException(error, request)) {
        await Future.delayed(retryPolicy!
            .delayRetryAttemptOnException(retryAttempt: retryCount + 1));
        return _attemptRequest(request, retryCount: retryCount + 1);
      } else {
        rethrow;
      }
    }

    return response;
  }

I also removed the global _retryCount variable.

I tried to open a PR to fix this on the master branch but I noticed the project is taken over by AI Slop and the code was very different between what's published on pub ( 2.0.0 ) and master.