Error when launching a job: return the error

[Sispheor]

Hello there 👋

When a job fail the current behavior is to only log and return None.

Would it be possible to return the error? Could be useful 😁

Thanks !

[Sispheor]

Maybe raising an exception here as we already have AuthFailed, SSLError, ConnectionError.

[costastf]

Hi @Sispheor , thanks for reporting. It is kind of a chosen pattern to return None in case of error in most of the codebase,
like can be seen here

towerlib/towerlib/towerlib.py

Line 264 in f522f53

if not response.ok:

and many other places. Raising an exception of course also makes sense but it means something completely different and would be a major change. I am not opposed to that just want to put this out there and see whether in other cases where the error is silent is actually the preferred response in which case we need to consider which way to go.

[Sispheor]

Maybe you can set this as config flag (raise_on_error) with default false to match the current behavior.