diff options
| author | Valentin Rothberg <rothberg@redhat.com> | 2021-09-23 13:35:57 +0200 |
|---|---|---|
| committer | Valentin Rothberg <rothberg@redhat.com> | 2021-09-23 16:07:54 +0200 |
| commit | f2c676e99ab6958aa5bf018d37a714be5807e4fe (patch) | |
| tree | 7227d408c1191205eed084b2b29dd451948365fc /pkg/domain/infra/abi | |
| parent | c58a677f8edc36c32728a87a6ac2c48d244d7ea2 (diff) | |
| download | podman-f2c676e99ab6958aa5bf018d37a714be5807e4fe.tar.gz podman-f2c676e99ab6958aa5bf018d37a714be5807e4fe.tar.bz2 podman-f2c676e99ab6958aa5bf018d37a714be5807e4fe.zip | |
[3.4] podman save: enforce signature removal
Enforce the removal of signatures in `podman save` to restore behavior
prior to the migration to libimage. We may consider improving on that
in the future. For details, please refer to the excellent summary by
@mtrmac [1].
[NO TESTS NEEDED] - manually verified but exisiting tests need some
further investigation (see [1]).
[1] https://github.com/containers/podman/pull/11669#issuecomment-925250264
Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
Diffstat (limited to 'pkg/domain/infra/abi')
| -rw-r--r-- | pkg/domain/infra/abi/images.go | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/pkg/domain/infra/abi/images.go b/pkg/domain/infra/abi/images.go index a88d38a10..aac9a2507 100644 --- a/pkg/domain/infra/abi/images.go +++ b/pkg/domain/infra/abi/images.go @@ -367,7 +367,10 @@ func (ir *ImageEngine) Load(ctx context.Context, options entities.ImageLoadOptio func (ir *ImageEngine) Save(ctx context.Context, nameOrID string, tags []string, options entities.ImageSaveOptions) error { saveOptions := &libimage.SaveOptions{} saveOptions.DirForceCompress = options.Compress - saveOptions.RemoveSignatures = options.RemoveSignatures + + // Force signature removal to preserve backwards compat. + // See https://github.com/containers/podman/pull/11669#issuecomment-925250264 + saveOptions.RemoveSignatures = true if !options.Quiet { saveOptions.Writer = os.Stderr |