HTTP Client custom retry function whether this situation will be abnormal

[bubble66]

Describe the Question

The http Client custom retry function will have an exception when canIdempotentRetry is false?
Reproducible Code

Expected behavior

	if canIdempotentRetry && client.DefaultRetryIf(req, resp, err) && errors.Is(err, errs.ErrBadPoolConn) {
		connAttempts++
		continue
	}

	if isDefaultRetryFunc {
		break
	}

	attempts++
	if attempts >= maxAttempts {
		break
	}

	// Check whether this request should be retried
	if canIdempotentRetry && !isRequestRetryable(req, resp, err) {
		break
	}

Would it be better to write like this?

[Duslia]

It seems it's a bug. Are you interested in raising a PR to fix it?

[welkeyever]

Seems it is by design - custom retry logic is process independently. Check this @li-jin-gou .