socket connection retry mechanism doesn't work in some situations

[weyou]

It looks like the socket connection retry mechanism doesn't work in some situations.

    def _conn_request(self, conn, request_uri, method, body, headers):
        i = 0
       ...
        while i < RETRIES:
            i += 1        # <------ here variable i becomes 1
            try:
                if conn.sock is None:
                    conn.connect()
                conn.request(method, request_uri, body, headers)
                ...
            except http.client.HTTPException:
                if conn.sock is None:
            ...

The global variable RETRIES is hardcoded into 2 currently. So the following code will never be executed since i always greater or equal than RETRIES - 1

                    if i < RETRIES - 1:
                        conn.close()
                        conn.connect()
                        continue
                    else:
                        conn.close()
                        raise
                if i < RETRIES - 1:
                    conn.close()
                    conn.connect()
                    continue

Maybe httplib2 allow user to change the value of httplib2.RETRIES. then above code block is reasonable.

But the second place:

            try:
                response = conn.getresponse()
            except :
            ...

The i always greater than 0, so the following code block will never be executed.

            except (socket.error, http.client.HTTPException):
                conn.close()
                if i == 0:
                    conn.close()
                    conn.connect()
                    continue

[temoto]

Yes, it is dead code. Thank you.

It's better in Python2 version

                if i < RETRIES - 1:

https://github.com/httplib2/httplib2/blob/master/python2/httplib2/__init__.py#L1603-L1610

[monluk]

Wouldn't it be better to remove "- 1" from the lines pointed in the bug report, so that there is "i < RETRIES"?
There are still people using that version and Python 2.