Block a user
feat(gitea): add retry logic for 5xx errors
Addressed MINOR findings from review round
Finding 1: Non-deterministic test (sonnet-review-bot)
Problem: TestDoGet_RetriesOnTemporaryNetError was timing-dependent with no…
feat(gitea): add retry logic for 5xx errors
Security Finding Addressed
Finding: Retry warnings log the full request URL and the last error, which could inadvertently leak sensitive query parameters if future callers pass them.
**Res…
feat(gitea): add retry logic for 5xx errors
Addressed Review Feedback (commit ac53ecf)
Addressed all MINOR findings from the previous review:
Fixed
1. isTemporaryNetError too broad (sonnet line ~277, gpt line ~234)
- Now…
feat(gitea): add retry logic for 5xx errors
Review Feedback Addressed (round 2)
Addressed the two MINOR findings from gpt-review-bot:
1. Configurable Backoff Durations
Added RetryBackoff field to Client struct:
- Defaults to…
feat(gitea): add retry logic for 5xx errors
Review Feedback Addressed
Fixed (commit 23da7ee)
- Dead backoff[0] element (sonnet-review + gpt-review NIT)
- Removed the unused
0duration from the backoff slice - Array now…
- Removed the unused
feat(gitea): add retry logic for 5xx errors
Self-review: PASS
No issues found — ready for human review.
Gates: All pass (make precommit succeeds)
Coherence: Single-purpose change — retry logic for 5xx errors. Every line…
feat(gitea): add retry logic for 5xx errors
Add retry logic for pattern fetch API calls