Repository navigation
chore(release): v1.7.0 - #50
Merged
Merged
Conversation
WithMaxRetryWait becomes a total wait budget (#42); WithTimeout stops mutating a caller-supplied http.Client (#38); WithHTTPClient(nil) no longer panics (#41); path escaping restored for spaces and percent-encoded '..' rejected (#40); the streaming reconnect budget resets after a healthy session (#34). Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_015ao5paex73xXvuM424Libo
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Bumps
Versioninclient.goto 1.7.0 and adds the[1.7.0]CHANGELOG section.What ships (since v1.6.0)
WithMaxRetryWaitbounds the total wait across retries, not each wait ([P3] Follow-up to #37: WithMaxRetryWait bounds one wait, not the total — 3x59s still blocks ~177s on context.Background() #42, PR fix: WithMaxRetryWait bounds one wait, not the total #48)WithTimeoutno longer mutates a caller-supplied*http.Client— data race and cross-client contamination ([P2] WithTimeout mutates a caller-supplied *http.Client — confirmed data race under -race #38, PR fix(client): isolate timeout overrides from shared HTTP clients #44)WithHTTPClient(nil)no longer panics on the first request ([P3] WithHTTPClient(nil) panics with a nil pointer dereference on the first request #41, PR fix: WithHTTPClient(nil) panics with a nil pointer dereference on the first request #45)..rejected ([P3] resolveEndpoint path contract: spaces over-rejected (regression), percent-encoded ".." under-rejected #40, PR fix: resolveEndpoint path contract — spaces over-rejected, percent-encoded ".." under-rejected #47)MaxReconnectAttemptscounts consecutive failures; budget resets after a healthy session ([P2][Coverage review] Reset streaming reconnect budget after defined healthy progress #34, PR fix: reset the streaming reconnect budget after a healthy session #49)Minor bump: the
WithMaxRetryWaitsemantic change tightens an upper bound and cannot lengthen any wait.Verification (local, this branch)
go vet ./...clean,gofmt -l .cleango test -race ./...→ ok1.6.0left outside CHANGELOGThe Go module proxy picks up the
v1.7.0tag after merge.🤖 Generated with Claude Code
https://claude.ai/code/session_015ao5paex73xXvuM424Libo