fix setting over URI

CheSema · CheSema · commit 83b2445831e7 · 2025-06-19T01:15:18.000+02:00
diff --git a/src/Server/HTTPHandler.cpp b/src/Server/HTTPHandler.cpp
@@ -450,7 +450,7 @@ void HTTPHandler::processQuery(
         {
             /// Other than query parameters are treated as settings.
             if (!customizeQueryParam(context, key, value))
-                settings_changes.push_back({key, value});
+                settings_changes.setSetting(key, value);
         }
     }
 
diff --git a/tests/queries/0_stateless/03545_settings_over_cgi_params.reference b/tests/queries/0_stateless/03545_settings_over_cgi_params.reference
@@ -5,4 +5,4 @@ output_format_parallel_formatting	0	1
 output_format_parallel_formatting=1&output_format_parallel_formatting=0
 output_format_parallel_formatting	0	1
 output_format_parallel_formatting=0&output_format_parallel_formatting=1
-output_format_parallel_formatting	0	1
+output_format_parallel_formatting	1	0

Original file line number	Diff line number	Diff line change
`@@ -450,7 +450,7 @@ void HTTPHandler::processQuery(`
`450`	`450`	`{`
`451`	`451`	`/// Other than query parameters are treated as settings.`
`452`	`452`	`if (!customizeQueryParam(context, key, value))`
`453`		`- settings_changes.push_back({key, value});`
	`453`	`+ settings_changes.setSetting(key, value);`
`454`	`454`	`}`
`455`	`455`	`}`
`456`	`456`