From 104fc7a12fcc4d262acb0dfa0e365e6aa0d0751e Mon Sep 17 00:00:00 2001 From: Ad Schellevis Date: Tue, 21 Jan 2025 16:30:08 +0100 Subject: [PATCH] net/relayd - regression in controller refactor (https://github.com/opnsense/core/issues/6389). As StatusController->toggleAction creates an instance of ServiceController, it needs to pass $request in order to check for isPost(). --- .../app/controllers/OPNsense/Relayd/Api/StatusController.php | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/net/relayd/src/opnsense/mvc/app/controllers/OPNsense/Relayd/Api/StatusController.php b/net/relayd/src/opnsense/mvc/app/controllers/OPNsense/Relayd/Api/StatusController.php index 40c1a3134..d2a8b0d5d 100644 --- a/net/relayd/src/opnsense/mvc/app/controllers/OPNsense/Relayd/Api/StatusController.php +++ b/net/relayd/src/opnsense/mvc/app/controllers/OPNsense/Relayd/Api/StatusController.php @@ -224,7 +224,9 @@ class StatusController extends ApiControllerBase $relaydMdl->serializeToConfig(); Config::getInstance()->save(); // invoke service controller - return (new ServiceController())->reconfigureAction(); + $srv = new ServiceController(); + $srv->request = $this->request; + return $srv->reconfigureAction(); } } return $result;