Skip to content

Implementes retries on failed requests#1

Merged
joente merged 3 commits into
mainfrom
retries
May 19, 2026
Merged

Implementes retries on failed requests#1
joente merged 3 commits into
mainfrom
retries

Conversation

@joente
Copy link
Copy Markdown
Contributor

@joente joente commented May 19, 2026

Description

In the real world, a website might not respond, just for a millisecond and we have to decide if we want to retry in such a case.
This also adds the retries (int) metric which indicates the number of retries used. (Should be 0 if the website could be reached, a value >0 means at least one retry was used to access the website.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

  • Test using alpha version

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • New and existing unit tests pass locally with my changes

@joente joente merged commit b6bb5cf into main May 19, 2026
4 checks passed
@joente joente deleted the retries branch May 19, 2026 10:05
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.

1 participant