Commit f16383f2 authored by smartcontracts's avatar smartcontracts Committed by GitHub

fix(cmn): legacy metrics binds to 0.0.0.0 (#2620)

Updates legacy MetricsServer to bind to 0.0.0.0 instead of 127.0.0.1 by
default which makes it possible to expose the metrics server from inside
docker. Only the DTL is still using legacy metrics, so should be fine.
Co-authored-by: default avatarmergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
parent 36cbfbf3
---
'@eth-optimism/common-ts': patch
---
Have legacy BaseService metrics bind to 0.0.0.0 by default
...@@ -56,7 +56,7 @@ export const createMetricsServer = async ( ...@@ -56,7 +56,7 @@ export const createMetricsServer = async (
}) })
const port = options.port || 7300 const port = options.port || 7300
const hostname = options.hostname || '127.0.0.1' const hostname = options.hostname || '0.0.0.0'
const server = app.listen(port, hostname, () => { const server = app.listen(port, hostname, () => {
logger.info('Metrics server started', { logger.info('Metrics server started', {
port, port,
......
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