Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve errors reported when pulling encrypted images without decryption #2646

Open
mtrmac opened this issue Nov 28, 2024 · 0 comments
Open
Labels
kind/feature A request for, or a PR adding, new functionality

Comments

@mtrmac
Copy link
Collaborator

mtrmac commented Nov 28, 2024

Right now, pulls of encrypted layers try to decrypt and extract them without handing encrypted data specially, so, we see something including invalid tar header (or, with future #2613, writing blob: layer 0 (blob "sha256:…"/""/"sha256:…") does not match config's DiffID "sha256:…").

That’s valuable for tests because we can ensure that the data really is encrypted, but bad for users.

We should add a “requires decrypted layers” field to private.ImageDestination, and abort a copy before starting to read any layers if isEncrypted and the transport requires decryption.

@mtrmac mtrmac added the kind/feature A request for, or a PR adding, new functionality label Nov 28, 2024
mtrmac added a commit to mtrmac/libpod that referenced this issue Nov 28, 2024
containers/image#2646 will track actually
returning a meaningful error instead of these internal details.

Signed-off-by: Miloslav Trmač <[email protected]>
mtrmac added a commit to mtrmac/libpod that referenced this issue Nov 28, 2024
containers/image#2646 will track actually
returning a meaningful error instead of these internal details.

Signed-off-by: Miloslav Trmač <[email protected]>
mtrmac added a commit to mtrmac/libpod that referenced this issue Nov 28, 2024
containers/image#2646 will track actually
returning a meaningful error instead of these internal details.

Signed-off-by: Miloslav Trmač <[email protected]>
mtrmac added a commit to mtrmac/libpod that referenced this issue Nov 29, 2024
containers/image#2646 will track actually
returning a meaningful error instead of these internal details.

Signed-off-by: Miloslav Trmač <[email protected]>
mtrmac added a commit to mtrmac/libpod that referenced this issue Nov 29, 2024
containers/image#2646 will track actually
returning a meaningful error instead of these internal details.

Signed-off-by: Miloslav Trmač <[email protected]>
mtrmac added a commit to mtrmac/libpod that referenced this issue Nov 29, 2024
containers/image#2646 will track actually
returning a meaningful error instead of these internal details.

Signed-off-by: Miloslav Trmač <[email protected]>
mtrmac added a commit to mtrmac/libpod that referenced this issue Dec 12, 2024
containers/image#2646 will track actually
returning a meaningful error instead of these internal details.

Signed-off-by: Miloslav Trmač <[email protected]>
mtrmac added a commit to mtrmac/libpod that referenced this issue Dec 14, 2024
containers/image#2646 will track actually
returning a meaningful error instead of these internal details.

Signed-off-by: Miloslav Trmač <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature A request for, or a PR adding, new functionality
Projects
None yet
Development

No branches or pull requests

1 participant