Skip to content

Commit

Permalink
microsoft.genpolicy: log image reference on unauthorized access
Browse files Browse the repository at this point in the history
  • Loading branch information
jmxnzo committed Dec 4, 2024
1 parent af71032 commit 0509b20
Show file tree
Hide file tree
Showing 2 changed files with 26 additions and 0 deletions.
Original file line number Diff line number Diff line change
@@ -0,0 +1,22 @@
From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
From: jmxnzo <[email protected]>
Date: Mon, 2 Dec 2024 12:38:04 +0100
Subject: [PATCH] genpolicy: include reference in logs when auth failure

---
src/tools/genpolicy/src/registry.rs | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/tools/genpolicy/src/registry.rs b/src/tools/genpolicy/src/registry.rs
index bdce2d40e3a7c3ec34137ceb3685fcc94aedcb39..9aa1cde98cd28a8c78d652986408e7738e5d96d6 100644
--- a/src/tools/genpolicy/src/registry.rs
+++ b/src/tools/genpolicy/src/registry.rs
@@ -125,7 +125,7 @@ impl Container {
})
}
Err(oci_distribution::errors::OciDistributionError::AuthenticationFailure(message)) => {
- panic!("Container image registry authentication failure ({}). Are docker credentials set-up for current user?", &message);
+ panic!("Container image registry authentication failure ({}) for {}. Are docker credentials set-up for current user?", &message, &reference.whole().as_str());
}
Err(e) => {
panic!(
4 changes: 4 additions & 0 deletions packages/by-name/microsoft/genpolicy/package.nix
Original file line number Diff line number Diff line change
Expand Up @@ -59,6 +59,10 @@ rustPlatform.buildRustPackage rec {
# to Microsofts genpolicy.
# TODO(miampf): remove when picked up by microsoft/kata-containers fork.
./0007-genpolicy-support-for-VOLUME-definition-in-container.patch

# Simple genpolicy logging patch to include the image reference in case of authentication failure
# Not merged, TODO(jmxnzo): remove when error logging was reworked or oci_distribution is updated to oci_client crate on microsoft/kata-containers fork.
./0008-genpolicy-include-reference-in-logs-when-auth-failure.patch
];
};

Expand Down

0 comments on commit 0509b20

Please sign in to comment.