autorest.DoRetryForStatusCodes infinitely retries on http.StatusTooManyRequests statuscode

[awesomenix]

Function autorest.DoRetryForStatusCode has this piece of code

// don't count a 429 against the number of attempts
				// so that we continue to retry until it succeeds
				if resp == nil || resp.StatusCode != http.StatusTooManyRequests {
					attempt++
				}

Basically infinitely retries in case of backoff, sometimes, though there is an opportunity for caller to cancel, the library should be flexible enough for client to provide a feature if the infinite retry feature is required.

[awesomenix]

Created PR: #323

[jhendrixMSFT]

Per discussion in #323 DoRetryForStatusCodes will be fixed to return the last response when the context is cancelled.

[jhendrixMSFT]

Fixed in https://github.com/Azure/go-autorest/releases/tag/v10.15.5.