Fix profiler in prod env

tchapi · tchapi · commit 4b7a5b2e9e25 · 2024-08-05T17:57:24.000+02:00
diff --git a/src/Controller/DAVController.php b/src/Controller/DAVController.php
@@ -291,10 +291,12 @@ private function initExceptionListener()
     }
 
     #[Route('/dav/{path}', name: 'dav', requirements: ['path' => '.*'])]
-    public function dav(Request $request, Profiler $profiler, string $path)
+    public function dav(Request $request, ?Profiler $profiler = null, ?string $path)
     {
         // We don't want the toolbar on the /dav/* routes
-        $profiler->disable();
+        if ($profiler instanceof Profiler) {
+            $profiler->disable();
+        }
 
         // We need to acknowledge the OPTIONS call before sabre/dav for public
         // calendars since we're circumventing the lib

Original file line number	Diff line number	Diff line change
`@@ -291,10 +291,12 @@ private function initExceptionListener()`
`291`	`291`	`}`
`292`	`292`
`293`	`293`	`#[Route('/dav/{path}', name: 'dav', requirements: ['path' => '.*'])]`
`294`		`- public function dav(Request $request, Profiler $profiler, string $path)`
	`294`	`+ public function dav(Request $request, ?Profiler $profiler = null, ?string $path)`
`295`	`295`	`{`
`296`	`296`	`// We don't want the toolbar on the /dav/* routes`
`297`		`- $profiler->disable();`
	`297`	`+ if ($profiler instanceof Profiler) {`
	`298`	`+ $profiler->disable();`
	`299`	`+ }`
`298`	`300`
`299`	`301`	`// We need to acknowledge the OPTIONS call before sabre/dav for public`
`300`	`302`	`// calendars since we're circumventing the lib`