Merge pull request #685 from tralexa/master
insecure flag not honored in cache
This commit is contained in:
		
						commit
						ccd99fbeec
					
				|  | @ -59,7 +59,7 @@ func (rc *RegistryCache) RetrieveLayer(ck string) (v1.Image, error) { | ||||||
| 	} | 	} | ||||||
| 
 | 
 | ||||||
| 	registryName := cacheRef.Repository.Registry.Name() | 	registryName := cacheRef.Repository.Registry.Name() | ||||||
| 	if rc.Opts.InsecureRegistries.Contains(registryName) { | 	if rc.Opts.Insecure || rc.Opts.InsecureRegistries.Contains(registryName) { | ||||||
| 		newReg, err := name.NewRegistry(registryName, name.WeakValidation, name.Insecure) | 		newReg, err := name.NewRegistry(registryName, name.WeakValidation, name.Insecure) | ||||||
| 		if err != nil { | 		if err != nil { | ||||||
| 			return nil, err | 			return nil, err | ||||||
|  |  | ||||||
|  | @ -77,6 +77,13 @@ func CheckPushPermissions(opts *config.KanikoOptions) error { | ||||||
| 		} | 		} | ||||||
| 
 | 
 | ||||||
| 		registryName := destRef.Repository.Registry.Name() | 		registryName := destRef.Repository.Registry.Name() | ||||||
|  | 		if opts.Insecure || opts.InsecureRegistries.Contains(registryName) { | ||||||
|  | 			newReg, err := name.NewRegistry(registryName, name.WeakValidation, name.Insecure) | ||||||
|  | 			if err != nil { | ||||||
|  | 				return errors.Wrap(err, "getting new insecure registry") | ||||||
|  | 			} | ||||||
|  | 			destRef.Repository.Registry = newReg | ||||||
|  | 		} | ||||||
| 		tr := makeTransport(opts, registryName) | 		tr := makeTransport(opts, registryName) | ||||||
| 		if err := remote.CheckPushPermission(destRef, creds.GetKeychain(), tr); err != nil { | 		if err := remote.CheckPushPermission(destRef, creds.GetKeychain(), tr); err != nil { | ||||||
| 			return errors.Wrapf(err, "checking push permission for %q", destRef) | 			return errors.Wrapf(err, "checking push permission for %q", destRef) | ||||||
|  |  | ||||||
		Loading…
	
		Reference in New Issue