apply retry decorator both on consumer class and consumer method

[mateochr]

Applying the retry decorator both on the consumer class and a consumer method results to stacking the number of retries (?). I would expect only the retry decorator of the consumer's method to be applied in such a case.

import logging

import uplink
from uplink.retry.when import status_5xx

logging.basicConfig()
logging.getLogger().setLevel(logging.DEBUG)


@uplink.retry(when=status_5xx(), max_attempts=4)
class Mock(uplink.Consumer):
    @uplink.retry(when=status_5xx(), max_attempts=2)
    @uplink.get("/500")
    def error(self):
        pass

    @uplink.get("/500")
    def error2(self):
        pass


client = Mock(base_url="https://mock.codes")

client.error()

DEBUG:urllib3.connectionpool:Starting new HTTPS connection (1): mock.codes:443
DEBUG:urllib3.connectionpool:https://mock.codes:443 "GET /500 HTTP/1.1" 500 None
DEBUG:urllib3.connectionpool:https://mock.codes:443 "GET /500 HTTP/1.1" 500 None
DEBUG:urllib3.connectionpool:https://mock.codes:443 "GET /500 HTTP/1.1" 500 None
DEBUG:urllib3.connectionpool:https://mock.codes:443 "GET /500 HTTP/1.1" 500 None
DEBUG:urllib3.connectionpool:https://mock.codes:443 "GET /500 HTTP/1.1" 500 None
DEBUG:urllib3.connectionpool:https://mock.codes:443 "GET /500 HTTP/1.1" 500 None
DEBUG:urllib3.connectionpool:https://mock.codes:443 "GET /500 HTTP/1.1" 500 None
DEBUG:urllib3.connectionpool:https://mock.codes:443 "GET /500 HTTP/1.1" 500 None

client.error2()
DEBUG:urllib3.connectionpool:https://mock.codes:443 "GET /500 HTTP/1.1" 500 None
DEBUG:urllib3.connectionpool:https://mock.codes:443 "GET /500 HTTP/1.1" 500 None
DEBUG:urllib3.connectionpool:https://mock.codes:443 "GET /500 HTTP/1.1" 500 None
DEBUG:urllib3.connectionpool:https://mock.codes:443 "GET /500 HTTP/1.1" 500 None

I'm not sure if that's the intended behavior here or if the same thing happens to other decorators.

uplink==0.9.7
python==3.8.10