From b42dfbf2a769af49d4a3175c65f0057404d01515 Mon Sep 17 00:00:00 2001 From: Bharath Gajjala <120367134+bgajjala8@users.noreply.github.com> Date: Tue, 3 Feb 2026 11:15:21 -0500 Subject: [PATCH] Adding error checks for set call --- internal/auth/oidc/service_callback.go | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/internal/auth/oidc/service_callback.go b/internal/auth/oidc/service_callback.go index bde7ff732f..d9f869773e 100644 --- a/internal/auth/oidc/service_callback.go +++ b/internal/auth/oidc/service_callback.go @@ -229,7 +229,11 @@ func Callback( val, err := pointerstructure.Get(evalData, searchPath) if err == nil { if str, ok := val.(string); ok { - pointerstructure.Set(evalData, searchPath, []string{str}) + if _, err := pointerstructure.Set(evalData, searchPath, []string{str}); err != nil { + // This should never happen, if we are able to Get it, + // but we validate anyways + return "", errors.Wrap(ctx, err, op) + } } } }