Logic in rest_client.py is incorrect

The code in rest_client.py related to 413 error code seems to be incorrect. Following is the logic when return code for request is 413:

if resp.status == 413:
            resp_body = self._parse_resp(resp_body)
            self._log(req_url, body, resp, resp_body)
            if 'overLimit' in resp_body:
                raise exceptions.OverLimit(resp_body['overLimit']['message'])
            elif 'limit' in resp_body['message']:
                raise exceptions.OverLimit(resp_body['message'])
            elif depth < MAX_RECURSION_DEPTH:
                delay = resp['Retry-After'] if 'Retry-After' in resp else 60
                time.sleep(int(delay))
                return self.request(method, url, headers, body, depth + 1)
            else:
                raise exceptions.RateLimitExceeded(
                    message=resp_body['overLimitFault']['message'],
                    details=resp_body['overLimitFault']['details'])

I understand the first two conditions ['overLimit' in resp_body] and ['limit' in resp_body['message']] are for the sake of confirming if the 413 code is due to “Absolute Limit exception”. But, even when rate-limit exception is encountered, the response body has word “rate-limited” in the message (implies word limit is present in message).

Thus No Retry is occurring when rate-limit is encountered. The flow gets stopped at second condition itself.

Hence removal of condition second condition (that checks "limit"word in response mesage) would be appropriate. Due to this logic in rest_client.py, tests script that encounters rate-limit fails in the first attempt without re-try.