+1


On Tue, Oct 22, 2013 at 12:04 PM, Michael Crosby
<notifications@github.com>wrote:

> In registry/registry.go:
>
> > @@ -362,7 +373,9 @@ func (r *Registry) PushImageChecksumRegistry(imgData *ImgData, registry string,
> >  	if err != nil {
> >  		return err
> >  	}
> > -	req.Header.Set("Authorization", "Token "+strings.Join(token, ","))
> > +	if req.Header.Get("Authorization") == "" { // Don't override
>
> This logic is duplicated a few times. I think you can factor this out into
> a func
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/dotcloud/docker/pull/2339/files#r7136641>
> .
>



-- 
Victor VIEUX
http://vvieux.com