frizbee icon indicating copy to clipboard operation
frizbee copied to clipboard

Add logging for update fails

Open amouat opened this issue 1 year ago • 3 comments

At the moment it seems Frizbee will just fail silently if it can't get a digest e.g. because of auth or network issues. I think it would be worth logging this to stderr.

For example:

FROM redis:invalid_image_tag

is clearly an image reference, but will just be ignored.

I guess this is the place in the code?

https://github.com/stacklok/frizbee/blob/main/pkg/replacer/replacer.go#L303

amouat avatar Jun 04 '24 15:06 amouat

hey, @amouat 👋 Thanks for reaching out!

You're right, logging that message might be handy 👍 Feel free to open a PR in case you already have one 😃

rdimitrov avatar Jun 04 '24 19:06 rdimitrov

Hi @amouat I'm reaching out to check if this is still relevant to you. We gladly accept contributions that improve the user experience, so feel free to open a PR if you want!

blkt avatar Oct 29 '24 16:10 blkt

I still think it makes sense, yes.

I'm not actively using frizbee right now, so I don't have a PR.

amouat avatar Oct 30 '24 09:10 amouat