From 4cda8669a51910060e737561ca5cb208f6383956 Mon Sep 17 00:00:00 2001 From: Bruno Date: Mon, 3 Nov 2025 12:22:36 -0300 Subject: [PATCH] Caching: GetKey requires a namespace argument (#113180) * Caching: GetKey requires a namespace argument * GetKey: special case empty namespace --- pkg/services/caching/service.go | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/pkg/services/caching/service.go b/pkg/services/caching/service.go index ad494bcaee4..c5bc577b122 100644 --- a/pkg/services/caching/service.go +++ b/pkg/services/caching/service.go @@ -84,7 +84,7 @@ func (s *OSSCachingService) HandleResourceRequest(ctx context.Context, req *back var _ CachingService = &OSSCachingService{} // GetKey creates a prefixed cache key and uses the internal `encoder` to encode the query into a string -func GetKey(prefix string, query interface{}) (string, error) { +func GetKey(namespace, prefix string, query interface{}) (string, error) { keybuf := bytes.NewBuffer(nil) encoder := &JSONEncoder{} @@ -98,6 +98,12 @@ func GetKey(prefix string, query interface{}) (string, error) { return "", err } + // The namespace is empty only when this function is used by the legacy caching module. + // This case can be removed when the legacy caching module is not being used anymore. + if namespace != "" { + return strings.Join([]string{namespace, prefix, key}, ":"), nil + } + return strings.Join([]string{prefix, key}, ":"), nil }