Safely copy paths to new *bo.Options (#672)

* Safely copy paths to new *bo.Options Signed-off-by: jakenichols2719 <[email protected]> * Add request_params to path members for SetDefaults Signed-off-by: jakenichols2719 <[email protected]> * Deleted log--not sure how it got there to begin with Signed-off-by: jakenichols2719 <[email protected]> --------- Signed-off-by: jakenichols2719 <[email protected]>
trickstercache · Dec 2, 2023 · 6eb6269 · 6eb6269
1 parent 961160a
commit 6eb6269
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/pkg/backends/options/options.go b/pkg/backends/options/options.go
@@ -633,6 +633,9 @@ func SetDefaults(
 		if err != nil {
 			return nil, err
 		}
+		for k, v := range o.Paths {
+			no.Paths[k] = v.Clone()
+		}
 	}
 
 	if metadata.IsDefined("backends", name, "alb") {

diff --git a/pkg/proxy/paths/options/options.go b/pkg/proxy/paths/options/options.go
@@ -193,7 +193,7 @@ func (o *Options) Merge(o2 *Options) {
 }
 
 var pathMembers = []string{"path", "match_type", "handler", "methods", "cache_key_params",
-	"cache_key_headers", "default_ttl_ms", "request_headers", "response_headers",
+	"cache_key_headers", "default_ttl_ms", "request_params", "request_headers", "response_headers",
 	"response_headers", "response_code", "response_body", "no_metrics", "collapsed_forwarding",
 	"req_rewriter_name",
 }