Comment 4 for bug 1826382

Revision history for this message
Liam Young (gnuoy) wrote :

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.