Merge pull request #512 from newrelic/vince/live-view-nil-uri

Handle when LiveView meta.uri is nil

Author: tpitale

lib/new_relic/telemetry/phoenix_live_view.ex

@@ -78,19 +78,19 @@ defmodule NewRelic.Telemetry.PhoenixLiveView do
 
   def handle_event(@live_view_mount_start, meas, meta, config) do
     if meta.socket.transport_pid do
-      uri = URI.parse(meta.uri)
+      {host, path} = parse_uri(meta.uri)
 
       # We're in the LiveView WebSocket process, collect a Transaction
-      with :collect <- Transaction.Reporter.start_transaction(:web, uri.path) do
+      with :collect <- Transaction.Reporter.start_transaction(:web, path) do
         NewRelic.DistributedTrace.start(:other)
         framework_name = "/Phoenix.LiveView/Live/#{inspect(meta.socket.view)}/#{meta.socket.assigns.live_action}"
 
         NewRelic.add_attributes(
           pid: inspect(self()),
           start_time: meas.system_time,
           framework_name: framework_name,
-          host: uri.host,
-          path: uri.path,
+          host: host,
+          path: path,
           "live_view.router": inspect(meta.socket.router),
           "live_view.endpoint": inspect(meta.socket.endpoint),
           "live_view.action": meta.socket.assigns[:live_action],
@@ -173,4 +173,12 @@ defmodule NewRelic.Telemetry.PhoenixLiveView do
   def handle_event(_event, _meas, _meta, _config) do
     :ignore
   end
+
+  defp parse_uri(nil),
+    do: {"unknown", "unknown"}
+
+  defp parse_uri(uri) do
+    uri = URI.parse(uri)
+    {uri.host, uri.path}
+  end
 end