Hi Matt,
Thank you for taking the time to triage the bug. I have tried out the decorator approach you mentioned ( https://paste.ubuntu.com/p/PDtwdNrqbx/ ) and it seems to work well. I am recreating a client whenever a 400 is returned because parsing the error string looking for the http/https error message looked brittle (and Apache specific). Do you think that's ok ?
I did also look at checking whether the endpoint has changed on error by creating a new client and checking its endpoint against self._client.get_endpoint() but that felt expensive.
I'd be really gratefully for any feedback here or I could work up some unit tests and submit a patch and we could continue the discussion there.
Hi Matt, /paste. ubuntu. com/p/PDtwdNrqb x/ ) and it seems to work well. I am recreating a client whenever a 400 is returned because parsing the error string looking for the http/https error message looked brittle (and Apache specific). Do you think that's ok ?
Thank you for taking the time to triage the bug. I have tried out the decorator approach you mentioned ( https:/
I did also look at checking whether the endpoint has changed on error by creating a new client and checking its endpoint against self._client. get_endpoint( ) but that felt expensive.
I'd be really gratefully for any feedback here or I could work up some unit tests and submit a patch and we could continue the discussion there.