-
Notifications
You must be signed in to change notification settings - Fork 8
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
cli: genpolicy logging: Add debug log level and repository reference to auth failure #1044
Conversation
packages/by-name/kata/kata-runtime/0017-Include-reference-in-logs-when-auth-failure.patch
Outdated
Show resolved
Hide resolved
24984c5
to
c4f9bc9
Compare
I guess it should also be possible to upstream that patch? |
c4f9bc9
to
e94bb43
Compare
I'm going to open an upstream issue and then finish the missing questions in the comment. |
e94bb43
to
ab6269d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
commit messages should be:
cli/genpolicy: include log-level 'debug' in Contrast logs
microsoft.genpolicy: log image reference on unauthorized access
c513151
to
583466c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
583466c
to
0509b20
Compare
0509b20
to
a75af8a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, I think the message is clear now!
The PR includes the debug log level of genpolicy into the Contrast CLI. Thus the resulting mapping is:
Contrast info == Genpolicy warning
Contrast debug == Genpolicy info/debug
As well it patches the existing genpolicy error message in case of an authentication failure, to include the reference to the repository.