Skip to content

check if rpcs and indexer_networks are aligned, log #750

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
paulo-ocean merged 3 commits into from
Nov 29, 2024
Merged

check if rpcs and indexer_networks are aligned, log #750

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
700b675
check if rpcs and indexer_networks are aligned, log
paulo-ocean Nov 12, 2024
20a41a8
improve log msg if network env vars mismatch
paulo-ocean Nov 13, 2024
af3161c
Merge branch 'main' into issue-728-misleading-msg
paulo-ocean Nov 25, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
21 changes: 18 additions & 3 deletions src/utils/config.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -102,9 +102,12 @@ function getSupportedChains(): RPCS | null {

function getIndexingNetworks(supportedNetworks: RPCS): RPCS | null {
const indexerNetworksEnv = process.env.INDEXER_NETWORKS

const defaultErrorMsg =
'Missing or invalid "INDEXER_NETWORKS" variable. Running Indexer with all supported networks defined in RPCS env variable...'
if (!indexerNetworksEnv) {
CONFIG_LOGGER.logMessageWithEmoji(
'INDEXER_NETWORKS is not defined, running Indexer with all supported networks defined in RPCS env variable ...',
defaultErrorMsg,
true,
GENERIC_EMOJIS.EMOJI_CHECK_MARK,
LOG_LEVELS_STR.LEVEL_INFO
Expand All @@ -114,9 +117,10 @@ function getIndexingNetworks(supportedNetworks: RPCS): RPCS | null {
try {
const indexerNetworks: number[] = JSON.parse(indexerNetworksEnv)

// env var exists but is wrong, so it does not index anything, but we still log the error
if (indexerNetworks.length === 0) {
CONFIG_LOGGER.logMessageWithEmoji(
'INDEXER_NETWORKS is an empty array, Running node without the Indexer component...',
'"INDEXER_NETWORKS" is an empty array, Running node without the Indexer component...',
true,
GENERIC_EMOJIS.EMOJI_CROSS_MARK,
LOG_LEVELS_STR.LEVEL_ERROR
Expand All @@ -132,10 +136,21 @@ function getIndexingNetworks(supportedNetworks: RPCS): RPCS | null {
return acc
}, {})

// if variables are not aligned we might end up not running indexer at all, so at least we should log a warning
if (Object.keys(filteredNetworks).length === 0) {
CONFIG_LOGGER.logMessageWithEmoji(
`"RPCS" chains: "${Object.keys(
supportedNetworks
)}" and "INDEXER_NETWORKS" chains: "${indexerNetworks}" mismatch! Running node without the Indexer component...`,
true,
GENERIC_EMOJIS.EMOJI_CROSS_MARK,
LOG_LEVELS_STR.LEVEL_ERROR
)
}
return filteredNetworks
} catch (e) {
CONFIG_LOGGER.logMessageWithEmoji(
'Missing or Invalid INDEXER_NETWORKS env variable format,running Indexer with all supported networks defined in RPCS env variable ...',
defaultErrorMsg,
true,
GENERIC_EMOJIS.EMOJI_CROSS_MARK,
LOG_LEVELS_STR.LEVEL_ERROR
Expand Down
Loading