containers / image

Work with containers' images
Apache License 2.0
862 stars 377 forks source link

Add validation error to digesting reader #2312

Closed saschagrunert closed 7 months ago

saschagrunert commented 7 months ago

The reason is hidden if expectedDigest.Validate() errors. This patch will add the error message to the stack to provide more details.

PTAL @containers/image-maintainers

saschagrunert commented 7 months ago

I’m curious, did this happen in practice somewhere?

No, just saw that the error got unused.

I also changed the error message below to align with the new style.

rhatdan commented 7 months ago

LGTM