resty icon indicating copy to clipboard operation
resty copied to clipboard

Change noRetryError for temporary errors

Open lavoiesl opened this issue 4 years ago • 1 comments

  • Expose an Unwrap method to leverage go's standard unwrapping
  • Let the error escape, such that hooks and end-users can access the Temporary information.
  • Do not wrap if the error already exposes Temporary, to honour it.

The main benefit is to leverage net/*'s standard errors; most of them already expose Temporary methods.

lavoiesl avatar Oct 18 '21 14:10 lavoiesl

@lavoiesl I like your proposal of using errors advantages introduced in go1.13. Currently, resty has minimum support as go1.11. I want to park this improvement to Resty v3 release.

jeevatkm avatar Oct 24 '21 23:10 jeevatkm