HttpServiceClientHandler throws after deciding to retry address

[PossiblyMax]

Just randomly browsing your repo, and this caught my eye and I thought I would raise it as a question.

In the file HttpServiceClientHandler.cs:

if (we.Status == WebExceptionStatus.ProtocolError)
{
    if (errorResponse.StatusCode == HttpStatusCode.NotFound ||
        errorResponse.StatusCode == HttpStatusCode.ServiceUnavailable)
    {
        // This could either mean we requested an endpoint that does not exist in the service API (a user error)
        // or the address that was resolved by fabric client is stale (transient runtime error) in which we should re-resolve.
        resolveAddress = true;
    }

    // On any other HTTP status codes, re-throw the exception to the caller.
    throw;
}

It looks like it wants to retry to resolve the address in the 'if' case, and sets the flag appropriately to do so, but then immediately throws the error back to the client, ignoring the retry. It appears the throw is expected to be in an else, so I thought I might mention it.