vbatts/containerd
1
0
Fork 0
Code Pull requests Releases Activity

Merge pull request #572 from stevvooe/fetcher-concurrency-bug

Browse source 
cmd/dist/fetch: address subtle concurrency bug
This commit is contained in:
Stephen Day 2017-02-24 18:41:38 -08:00 committed by GitHub
commit c2c057a5db
1 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
cmd/dist/fetch.go vendored
View file

@ -139,11 +139,11 @@ func getResolver(ctx contextpkg.Context) (remotes.Resolver, error) {
} }
for _, path := range paths { for _, path := range paths {
base.Path = path url := base
url := base.String() url.Path = path
log.G(ctx).WithField("url", url).Debug("fetch content") log.G(ctx).WithField("url", url).Debug("fetch content")
req, err := http.NewRequest(http.MethodGet, url, nil) req, err := http.NewRequest(http.MethodGet, url.String(), nil)
if err != nil { if err != nil {
return nil, err return nil, err
} }