Skip to content

Commit

Permalink
fix: metrics-host-allowlist for Teku (#765)
Browse files Browse the repository at this point in the history
Currently `'*'` is passed to Teku, which is not correct, so I see `Not
Authorized` when trying to access Teku metrics, Prometheus says that
Teku is down because of 403, and Grafana metrics are empty.
Only `*` should be passed, which is the purpose of the fix.
I don't know how to test it, so not 100% on the fix.
  • Loading branch information
zilm13 authored Sep 16, 2024
1 parent 6f868cc commit ad75fcc
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/cl/teku/teku_launcher.star
Original file line number Diff line number Diff line change
Expand Up @@ -295,7 +295,7 @@ def get_beacon_config(
# vvvvvvvvvvvvvvvvvvv METRICS CONFIG vvvvvvvvvvvvvvvvvvvvv
"--metrics-enabled",
"--metrics-interface=0.0.0.0",
"--metrics-host-allowlist='*'",
"--metrics-host-allowlist=*",
"--metrics-categories=BEACON,PROCESS,LIBP2P,JVM,NETWORK,PROCESS",
"--metrics-port={0}".format(BEACON_METRICS_PORT_NUM),
# ^^^^^^^^^^^^^^^^^^^ METRICS CONFIG ^^^^^^^^^^^^^^^^^^^^^
Expand Down

0 comments on commit ad75fcc

Please sign in to comment.