Skip to content

Commit 10d1923

Browse files
authored
Merge pull request #5834 from tonistiigi/fix-user-agent
resolver: fix missing user-agent on layer requests
2 parents ec9022a + cfb8d40 commit 10d1923

File tree

1 file changed

+7
-3
lines changed

1 file changed

+7
-3
lines changed

util/resolver/pool.go

+7-3
Original file line numberDiff line numberDiff line change
@@ -125,14 +125,16 @@ func newResolver(hosts docker.RegistryHosts, handler *authHandlerNS, sm *session
125125
docker.WithPlainHTTP(docker.MatchLocalhost),
126126
)
127127
}
128+
headers := http.Header{}
129+
headers.Set("User-Agent", version.UserAgent())
128130
r := &Resolver{
129131
hosts: hosts,
130132
sm: sm,
131133
g: g,
132134
handler: handler,
135+
headers: headers,
133136
}
134-
headers := http.Header{}
135-
headers.Set("User-Agent", version.UserAgent())
137+
136138
r.Resolver = docker.NewResolver(docker.ResolverOptions{
137139
Hosts: r.HostsFunc,
138140
Headers: headers,
@@ -144,6 +146,7 @@ func newResolver(hosts docker.RegistryHosts, handler *authHandlerNS, sm *session
144146
type Resolver struct {
145147
remotes.Resolver
146148
hosts docker.RegistryHosts
149+
headers http.Header
147150
sm *session.Manager
148151
g session.Group
149152
handler *authHandlerNS
@@ -196,7 +199,8 @@ func (r *Resolver) WithSession(s session.Group) *Resolver {
196199
r2.auth = nil
197200
r2.g = s
198201
r2.Resolver = docker.NewResolver(docker.ResolverOptions{
199-
Hosts: r2.HostsFunc, // this refers to the newly-configured session so we need to recreate the resolver.
202+
Hosts: r2.HostsFunc, // this refers to the newly-configured session so we need to recreate the resolver.
203+
Headers: r2.headers.Clone(),
200204
})
201205
return &r2
202206
}

0 commit comments

Comments
 (0)