zmb3 / spotify

A Go wrapper for the Spotify Web API
Apache License 2.0
1.4k stars 295 forks source link

added maxRetryDuration option to client #273

Open Pineapple217 opened 1 week ago

Pineapple217 commented 1 week ago

See #263

Pineapple217 commented 1 week ago

Thanks for the fixes @flimzy

Pineapple217 commented 1 week ago

I could add custom error type to expose the retry duration to the user. Or should this lib stay minimal?

flimzy commented 1 week ago

I could add custom error type to expose the retry duration to the user. Or should this lib stay minimal?

I just saw this, after I created a PR to do exactly that. :laughing:

IMO, exposing that is quite important. Let's see what the maintainers say.