Skip to content

Commit 5c89400

Browse files
authored
hotfix(telemetry): do not discard operators without metadata (#1803)
1 parent 50a64c6 commit 5c89400

File tree

2 files changed

+17
-0
lines changed

2 files changed

+17
-0
lines changed

telemetry_api/lib/telemetry_api/operators.ex

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,6 @@
11
defmodule TelemetryApi.Operators do
2+
require Logger
3+
24
@moduledoc """
35
The Operators context.
46
"""
@@ -130,6 +132,7 @@ defmodule TelemetryApi.Operators do
130132
# {:error, string}
131133
#
132134
defp add_operator_metadata(op_data) do
135+
Logger.info("Fetching metadata for operator: #{op_data.address}")
133136
with {:ok, url} <- DelegationManager.get_operator_url(op_data.address),
134137
{:ok, metadata} <- TelemetryApi.Utils.fetch_json_data(url) do
135138
operator = %{
@@ -140,6 +143,17 @@ defmodule TelemetryApi.Operators do
140143
}
141144

142145
{:ok, operator}
146+
else
147+
{:error, reason} ->
148+
Logger.error("Failed to fetch metadata for operator: #{op_data.address}. Reason: #{inspect(reason)}")
149+
operator = %{
150+
id: op_data.id,
151+
address: op_data.address,
152+
stake: op_data.stake,
153+
name: op_data.address
154+
}
155+
156+
{:ok, operator}
143157
end
144158
end
145159

telemetry_api/lib/telemetry_api/utils.ex

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,6 @@
11
defmodule TelemetryApi.Utils do
2+
require Logger
3+
24
@moduledoc """
35
Some utility functions
46
"""
@@ -15,6 +17,7 @@ defmodule TelemetryApi.Utils do
1517
{:error, message}
1618
"""
1719
def fetch_json_data(url) do
20+
Logger.info("Fetching data from #{url}")
1821
with {:ok, %HTTPoison.Response{status_code: 200, body: body}} <- HTTPoison.get(url) do
1922
Jason.decode(body)
2023
else

0 commit comments

Comments
 (0)