Commit ad75fcce authored by Dmitrii Shmatko's avatar Dmitrii Shmatko Committed by GitHub

fix: `metrics-host-allowlist` for Teku (#765)

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.
parent 6f868ccf
...@@ -295,7 +295,7 @@ def get_beacon_config( ...@@ -295,7 +295,7 @@ def get_beacon_config(
# vvvvvvvvvvvvvvvvvvv METRICS CONFIG vvvvvvvvvvvvvvvvvvvvv # vvvvvvvvvvvvvvvvvvv METRICS CONFIG vvvvvvvvvvvvvvvvvvvvv
"--metrics-enabled", "--metrics-enabled",
"--metrics-interface=0.0.0.0", "--metrics-interface=0.0.0.0",
"--metrics-host-allowlist='*'", "--metrics-host-allowlist=*",
"--metrics-categories=BEACON,PROCESS,LIBP2P,JVM,NETWORK,PROCESS", "--metrics-categories=BEACON,PROCESS,LIBP2P,JVM,NETWORK,PROCESS",
"--metrics-port={0}".format(BEACON_METRICS_PORT_NUM), "--metrics-port={0}".format(BEACON_METRICS_PORT_NUM),
# ^^^^^^^^^^^^^^^^^^^ METRICS CONFIG ^^^^^^^^^^^^^^^^^^^^^ # ^^^^^^^^^^^^^^^^^^^ METRICS CONFIG ^^^^^^^^^^^^^^^^^^^^^
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment