[trafficserver] branch 9.0.x updated: Adjust to actually try a server address more than once (#7288)

[zw...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

zwoop pushed a commit to branch 9.0.x
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/9.0.x by this push:
     new c3ecb30  Adjust to actually try a server address more than once (#7288)
c3ecb30 is described below

commit c3ecb309d8b9afebc3cb8c85ac0b9a5190723a25
Author: Susan Hinrichs <sh...@yahoo-inc.com>
AuthorDate: Mon Feb 1 13:04:24 2021 -0600

    Adjust to actually try a server address more than once (#7288)
    
    (cherry picked from commit dc2dbb09a6228ed5c4da776952de2eda123f2275)
---
 proxy/http/HttpTransact.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/proxy/http/HttpTransact.cc b/proxy/http/HttpTransact.cc
index e4db42e..a6ad9f9 100644
--- a/proxy/http/HttpTransact.cc
+++ b/proxy/http/HttpTransact.cc
@@ -3779,7 +3779,7 @@ HttpTransact::handle_response_from_server(State *s)
         return CallOSDNSLookup(s);
       } else if ((s->dns_info.srv_lookup_success || s->host_db_info.is_rr_elt()) &&
                  (s->txn_conf->connect_attempts_rr_retries > 0) &&
-                 (s->current.attempts % s->txn_conf->connect_attempts_rr_retries == 0)) {
+                 ((s->current.attempts + 1) % s->txn_conf->connect_attempts_rr_retries == 0)) {
         delete_server_rr_entry(s, max_connect_retries);
         return;
       } else {