fix empty route

eeliu · eeliu · commit 329f7bf0f834 · 2024-09-02T11:14:50.000+08:00
diff --git a/lib/Pinpoint/Plugins/LaravelPerRequest.php b/lib/Pinpoint/Plugins/LaravelPerRequest.php
@@ -21,8 +21,17 @@ class LaravelPerRequest extends DefaultRequestPlugin
 {
     public function __destruct()
     {
-        pinpoint_set_context(PP_ROUTE_KEY, Request()->route()->uri());
+        $route = Request()->route();
+        if ($route) {
+            pinpoint_set_context(PP_ROUTE_KEY, Request()->route()->uri());
+        } else {
+            pinpoint_set_context(PP_ROUTE_KEY, "_none_");
+        }
         parent::__destruct();
     }
 }
-//author: eeliu
+//author: eeliu
+
+// Changes
+// 2024/9/02
+// fix "Request()->route()" can be none

Original file line number	Diff line number	Diff line change
`@@ -21,8 +21,17 @@ class LaravelPerRequest extends DefaultRequestPlugin`
`21`	`21`	`{`
`22`	`22`	`public function __destruct()`
`23`	`23`	`{`
`24`		`- pinpoint_set_context(PP_ROUTE_KEY, Request()->route()->uri());`
	`24`	`+ $route = Request()->route();`
	`25`	`+ if ($route) {`
	`26`	`+ pinpoint_set_context(PP_ROUTE_KEY, Request()->route()->uri());`
	`27`	`+ } else {`
	`28`	`+ pinpoint_set_context(PP_ROUTE_KEY, "_none_");`
	`29`	`+ }`
`25`	`30`	`parent::__destruct();`
`26`	`31`	`}`
`27`	`32`	`}`
`28`		`-//author: eeliu`
	`33`	`+//author: eeliu`
	`34`	`+`
	`35`	`+// Changes`
	`36`	`+// 2024/9/02`
	`37`	`+// fix "Request()->route()" can be none`