Skip to content

[pull] master from DataDog:master#424

Merged
pull[bot] merged 3 commits into
ConnectionMaster:masterfrom
DataDog:master
Jun 29, 2026
Merged

[pull] master from DataDog:master#424
pull[bot] merged 3 commits into
ConnectionMaster:masterfrom
DataDog:master

Conversation

@pull

@pull pull Bot commented Jun 29, 2026

Copy link
Copy Markdown

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

Co-authored-by: ci.datadog-api-spec <packages@datadoghq.com>
* Fix zero-delay retries when HTTPClient.Timeout is unset

When no custom HTTPClient is provided, NewAPIClient falls back to
http.DefaultClient which has Timeout==0 (Go's "no timeout" sentinel).
The backoff cap treated 0 as zero seconds, causing math.Min(0, retryVal)
to always return 0 and all retries to fire instantly.

Guard the cap so it only applies when Timeout > 0. Also adds unit tests
covering both the zero-timeout and capped cases.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* pre-commit fixes

---------

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-authored-by: ci.datadog-api-spec <packages@datadoghq.com>
Co-authored-by: ci.datadog-api-spec <packages@datadoghq.com>
@pull pull Bot locked and limited conversation to collaborators Jun 29, 2026
@pull pull Bot added the ⤵️ pull label Jun 29, 2026
@pull pull Bot merged commit 7e3948f into ConnectionMaster:master Jun 29, 2026
1 check passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant