fix: add number of addresses considered in URL for resource error message (#282)

alanshaw · web-flow · commit 22b7f7a1fd68 · 2025-12-04T13:00:24.000Z
Allows better debugging, if there were 0 addresses returned for example
then we know it's a problem with the results...
diff --git a/pkg/service/service.go b/pkg/service/service.go
@@ -361,7 +361,7 @@ func urlForResource(provider peer.AddrInfo, replacements []replacement) (*url.UR
 		}
 	}
 	placeholders := strings.Join(slices.Collect(iterable.Map(func(r replacement) string { return r.resourcePlaceholder }, slices.Values(replacements))), " or ")
-	return nil, fmt.Errorf("no %s endpoint found", placeholders)
+	return nil, fmt.Errorf("no %s endpoint found in %d addresses", placeholders, len(provider.Addrs))
 }
 
 func fetchClaimURL(provider peer.AddrInfo, claimCid cid.Cid) (*url.URL, error) {

Original file line number	Diff line number	Diff line change
`@@ -361,7 +361,7 @@ func urlForResource(provider peer.AddrInfo, replacements []replacement) (*url.UR`
`361`	`361`	`}`
`362`	`362`	`}`
`363`	`363`	`placeholders := strings.Join(slices.Collect(iterable.Map(func(r replacement) string { return r.resourcePlaceholder }, slices.Values(replacements))), " or ")`
`364`		`- return nil, fmt.Errorf("no %s endpoint found", placeholders)`
	`364`	`+ return nil, fmt.Errorf("no %s endpoint found in %d addresses", placeholders, len(provider.Addrs))`
`365`	`365`	`}`
`366`	`366`
`367`	`367`	`func fetchClaimURL(provider peer.AddrInfo, claimCid cid.Cid) (*url.URL, error) {`