Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change timeout by a function #65

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

treussart
Copy link

Replace the Timeout field with a function, which will allow me to randomize the timeout to prevent all customers from retrying in the same time frame.

@jerome-laforge
Copy link

jerome-laforge commented Nov 13, 2024

I am not the maintainer but in order to not break the API according semver, I think that you must define TimeoutFunc func() time.Duration and let Timeout as-it. And if TimeoutFunc is not nil then use it instead of timeout else fallback on classic timeout behavior.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants