From d947b980e8fecd5127a053193d6fc0923af7c292 Mon Sep 17 00:00:00 2001 From: David Grove Date: Mon, 8 Sep 2025 14:35:59 -0400 Subject: [PATCH] Sanitize UserInfo.UID before using as label value Fixes #367 --- internal/webhook/appwrapper_webhook.go | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/internal/webhook/appwrapper_webhook.go b/internal/webhook/appwrapper_webhook.go index 4253166..bd4fce7 100644 --- a/internal/webhook/appwrapper_webhook.go +++ b/internal/webhook/appwrapper_webhook.go @@ -92,7 +92,8 @@ func (w *appWrapperWebhook) Default(ctx context.Context, obj runtime.Object) err } userInfo := request.UserInfo username := utils.SanitizeLabel(userInfo.Username) - aw.Labels = utilmaps.MergeKeepFirst(map[string]string{AppWrapperUsernameLabel: username, AppWrapperUserIDLabel: userInfo.UID}, aw.Labels) + userUID := utils.SanitizeLabel(userInfo.UID) + aw.Labels = utilmaps.MergeKeepFirst(map[string]string{AppWrapperUsernameLabel: username, AppWrapperUserIDLabel: userUID}, aw.Labels) return nil }