Merge pull request ActiveState#3393 from ActiveState/mitchell/dx-2947

mitchell-as · web-flow · commit 95740fab4454 · 2024-07-09T16:34:17.000-04:00
Do not use the default retryhttp client for `state update`; use a no-timeout client.
diff --git a/internal/updater/fetcher.go b/internal/updater/fetcher.go
@@ -26,7 +26,7 @@ type Fetcher struct {
 }
 
 func NewFetcher(an analytics.Dispatcher) *Fetcher {
-	return &Fetcher{retryhttp.DefaultClient, an}
+	return &Fetcher{retryhttp.NewClient(0, retryhttp.DefaultRetries), an}
 }
 
 func (f *Fetcher) Fetch(update *UpdateInstaller, targetDir string) error {

Original file line number	Diff line number	Diff line change
`@@ -26,7 +26,7 @@ type Fetcher struct {`
`26`	`26`	`}`
`27`	`27`
`28`	`28`	`func NewFetcher(an analytics.Dispatcher) *Fetcher {`
`29`		`- return &Fetcher{retryhttp.DefaultClient, an}`
	`29`	`+ return &Fetcher{retryhttp.NewClient(0, retryhttp.DefaultRetries), an}`
`30`	`30`	`}`
`31`	`31`
`32`	`32`	`func (f Fetcher) Fetch(update UpdateInstaller, targetDir string) error {`