Skip to content
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.

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

If RPCS and INDEXER_NETWORKS are not aligned we should show a log message #728

Open
paulo-ocean opened this issue Oct 22, 2024 · 1 comment · May be fixed by #750
Open

If RPCS and INDEXER_NETWORKS are not aligned we should show a log message #728

paulo-ocean opened this issue Oct 22, 2024 · 1 comment · May be fixed by #750
Assignees
Labels
Type: Enhancement New feature or request

Comments

@paulo-ocean
Copy link
Contributor

paulo-ocean commented Oct 22, 2024

INDEXER_NETWORKS takes precedence over RPCS...
if the INDEXER_NETWORKS contains a different chain than the ones defined on RPCS (or if the chains from RPCS are not present on INDEXER_NETWORKS)
It does not index anything, but there are no logs or anything warning about the .env mismatch..
at least a warning message would be nice, since the 2 .env vars if present, need to be aligned...
This can generate some confusion

@paulo-ocean paulo-ocean added the Type: Enhancement New feature or request label Oct 22, 2024
@paulo-ocean paulo-ocean self-assigned this Oct 22, 2024
@paulo-ocean
Copy link
Contributor Author

paulo-ocean commented Oct 22, 2024

additionally, this message should also be a warning, not an error:
2024-10-22T17:42:41.685Z error: CONFIG: ❌ Missing or Invalid INDEXER_NETWORKS env variable format,running Indexer with all supported networks defined in RPCS env variable ...
INDEXER_NETWORKS is NOT mandatory, and if we have RPCS its enough, so it should be a WARN at most, not an ERROR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant