decentraland / realm-provider / 31192497029

37%

main: 51%

Build Type

push

github

Committed by web-flow

Commit Message

fix: skip blacklisted catalysts before fetching /about (#87)

* fix: skip blacklisted catalysts before fetching /about

BLACKLISTED_CATALYST was already used in about-main-handler to filter
results after the fact, but getHealhtyCatalysts() still attempted to
connect to every DAO node — including shut-down ones like
peer.melonwave.com.

This caused a flood of AbortErrors on every health-check cycle, which
saturated the Node.js event loop and made the metrics endpoint on :5000
temporarily unresponsive, triggering the Prometheus realm-provider-ea-down
alert even though the ECS task itself was healthy.

The fix moves the blacklist filter to before the Promise.all fetch in
realm-provider.ts so blacklisted nodes are never contacted. Also enables
peer.melonwave.com in .env.default since that node has been shut down by
its owner.

* chore: retrigger CI after transient Docker Hub 500

The previous quay-build-push failed because Docker Hub returned a 500
when pulling the node:24-alpine base image. No code changes.

* fix: revert .env.default blacklist change

The BLACKLISTED_CATALYST value is managed in the definitions repo
(decentraland/definitions), not in the app defaults.

Coverage Stats

47 of 93 branches covered (50.54%)

Branch coverage included in aggregate %.

0 of 5 new or added lines in 1 file covered. (0.0%)

2 existing lines in 1 file now uncovered.

113 of 334 relevant lines covered (33.83%)

16.13 hits per line