search

sillsdev / libpalaso

Palaso Library: A set of .Net libraries useful for developers of Language Software.
MIT License
44 stars 51 forks source link

feat: Change default parameter value of `RetryUtility.Retry` exception list #1295

Closed ermshiperete closed 10 months ago

ermshiperete commented 10 months ago

It would be better if RetryUtility.Retry would retry on all exceptions instead of not retrying at all if exceptionTypesToRetry is not specified. We should change the default value to catch Exception.

The current implementation has a fairly unexpected behavior (not retrying if you use the default value).

See #1294

ermshiperete commented 10 months ago

We already catch IOException if the passed in parameter is null (see RetryUtility.Retry<T>).