Keep exception in case of error

[rintcius]

... also rename because we can't assume these are always auth errors

[rintcius]

Changing because of SRV-56
This is not intended to fix the issue yet - just want to propagate the underlying exception for now, so that we can see what's the actual problem

[domaspoliakas]

Is the intention to just gracefully error, or are we expecting to be recovering from these downstream as well?

[rintcius]

Is the intention to just gracefully error, or are we expecting to be recovering from these downstream as well?

We're intermittently seeing Error.Authentication failures. At least some of these don’t appear to be actual authentication issues - but since the underlying exception is currently swallowed, we don’t have more detail.

The plan is to run this change in prod for a while to log the underlying exceptions and gather more information. Based on what we find, we can adjust the handling accordingly - I expect some of these errors will be retriable.