Code review comment for ~nteodosio/software-properties:catch-exception

Revision history for this message
Nathan Teodosio (nteodosio) wrote :

I think printing the context is indeed a good idea, but unsure about
catching specific exceptions.

The UA API[1] does not even list TimeoutError, it is raised by SSL, so
maybe by trying to be more specific we might not exhaust the list of
possibilities?

[1]:
https://github.com/canonical/ubuntu-advantage-client/blob/main/docs/references/api.md#raised-exceptions-2

« Back to merge proposal