fix(gateway): allow slashes in configuration parameter IDs

  ROS2 parameters like qos_overrides./parameter_events.publisher.depth
  contain slashes in their names. cpp-httplib decodes percent-encoded
  URLs (%2F -> /), so the route regex must use (.+) instead of ([^/]+)
  to match the full parameter name.

  This is consistent with the existing pattern for topic routes.

Author: mfaferek93

src/ros2_medkit_gateway/src/http/rest_server.cpp

@@ -284,19 +284,21 @@ void RESTServer::setup_routes() {
            });
 
   // App configurations - get specific
-  srv->Get((api_path("/apps") + R"(/([^/]+)/configurations/([^/]+)$)"),
+  // Use (.+) for config_id to accept slashes from percent-encoded URLs (%2F -> /)
+  // ROS2 parameters like qos_overrides./parameter_events.publisher.depth contain slashes
+  srv->Get((api_path("/apps") + R"(/([^/]+)/configurations/(.+)$)"),
            [this](const httplib::Request & req, httplib::Response & res) {
              config_handlers_->handle_get_configuration(req, res);
            });
 
   // App configurations - set
-  srv->Put((api_path("/apps") + R"(/([^/]+)/configurations/([^/]+)$)"),
+  srv->Put((api_path("/apps") + R"(/([^/]+)/configurations/(.+)$)"),
            [this](const httplib::Request & req, httplib::Response & res) {
              config_handlers_->handle_set_configuration(req, res);
            });
 
   // App configurations - delete single
-  srv->Delete((api_path("/apps") + R"(/([^/]+)/configurations/([^/]+)$)"),
+  srv->Delete((api_path("/apps") + R"(/([^/]+)/configurations/(.+)$)"),
               [this](const httplib::Request & req, httplib::Response & res) {
                 config_handlers_->handle_delete_configuration(req, res);
               });
@@ -391,17 +393,17 @@ void RESTServer::setup_routes() {
              config_handlers_->handle_list_configurations(req, res);
            });
 
-  srv->Get((api_path("/functions") + R"(/([^/]+)/configurations/([^/]+)$)"),
+  srv->Get((api_path("/functions") + R"(/([^/]+)/configurations/(.+)$)"),
            [this](const httplib::Request & req, httplib::Response & res) {
              config_handlers_->handle_get_configuration(req, res);
            });
 
-  srv->Put((api_path("/functions") + R"(/([^/]+)/configurations/([^/]+)$)"),
+  srv->Put((api_path("/functions") + R"(/([^/]+)/configurations/(.+)$)"),
            [this](const httplib::Request & req, httplib::Response & res) {
              config_handlers_->handle_set_configuration(req, res);
            });
 
-  srv->Delete((api_path("/functions") + R"(/([^/]+)/configurations/([^/]+)$)"),
+  srv->Delete((api_path("/functions") + R"(/([^/]+)/configurations/(.+)$)"),
               [this](const httplib::Request & req, httplib::Response & res) {
                 config_handlers_->handle_delete_configuration(req, res);
               });
@@ -526,17 +528,17 @@ void RESTServer::setup_routes() {
              config_handlers_->handle_list_configurations(req, res);
            });
 
-  srv->Get((api_path("/areas") + R"(/([^/]+)/configurations/([^/]+)$)"),
+  srv->Get((api_path("/areas") + R"(/([^/]+)/configurations/(.+)$)"),
            [this](const httplib::Request & req, httplib::Response & res) {
              config_handlers_->handle_get_configuration(req, res);
            });
 
-  srv->Put((api_path("/areas") + R"(/([^/]+)/configurations/([^/]+)$)"),
+  srv->Put((api_path("/areas") + R"(/([^/]+)/configurations/(.+)$)"),
            [this](const httplib::Request & req, httplib::Response & res) {
              config_handlers_->handle_set_configuration(req, res);
            });
 
-  srv->Delete((api_path("/areas") + R"(/([^/]+)/configurations/([^/]+)$)"),
+  srv->Delete((api_path("/areas") + R"(/([^/]+)/configurations/(.+)$)"),
               [this](const httplib::Request & req, httplib::Response & res) {
                 config_handlers_->handle_delete_configuration(req, res);
               });
@@ -670,19 +672,19 @@ void RESTServer::setup_routes() {
            });
 
   // Get specific configuration (parameter) - register before general route
-  srv->Get((api_path("/components") + R"(/([^/]+)/configurations/([^/]+)$)"),
+  srv->Get((api_path("/components") + R"(/([^/]+)/configurations/(.+)$)"),
            [this](const httplib::Request & req, httplib::Response & res) {
              config_handlers_->handle_get_configuration(req, res);
            });
 
   // Set configuration (parameter)
-  srv->Put((api_path("/components") + R"(/([^/]+)/configurations/([^/]+)$)"),
+  srv->Put((api_path("/components") + R"(/([^/]+)/configurations/(.+)$)"),
            [this](const httplib::Request & req, httplib::Response & res) {
              config_handlers_->handle_set_configuration(req, res);
            });
 
   // Delete (reset) single configuration to default value
-  srv->Delete((api_path("/components") + R"(/([^/]+)/configurations/([^/]+)$)"),
+  srv->Delete((api_path("/components") + R"(/([^/]+)/configurations/(.+)$)"),
               [this](const httplib::Request & req, httplib::Response & res) {
                 config_handlers_->handle_delete_configuration(req, res);
               });

src/ros2_medkit_gateway/test/test_configuration_manager.cpp

@@ -378,7 +378,7 @@ TEST_F(TestConfigurationManager, test_concurrent_parameter_operations_no_executo
       for (int op = 0; op < kOpsPerThread; ++op) {
         try {
           // Mix different operations to stress test serialization
-          switch ((i + op) % 4) {
+          switch ((i + op) % 6) {
             case 0: {
               auto result = config_manager_->list_parameters("/test_config_manager_node");
               if (result.success) {
@@ -408,6 +408,22 @@ TEST_F(TestConfigurationManager, test_concurrent_parameter_operations_no_executo
               }
               break;
             }
+            case 4: {
+              auto result = config_manager_->reset_parameter("/test_config_manager_node", "concurrent_test_int");
+              if (result.success) {
+                success_count++;
+              }
+              break;
+            }
+            case 5: {
+              auto result = config_manager_->reset_all_parameters("/test_config_manager_node");
+              // reset_all_parameters returns success=false if some params are read-only,
+              // but data is still populated - count as success if no exception
+              if (result.success || result.data.contains("reset_count")) {
+                success_count++;
+              }
+              break;
+            }
           }
         } catch (const std::exception & e) {
           // This should NOT happen - executor conflicts would throw here