Add check for chain ID

[mariacarmina]

Description

Add check for chain ID. Added a new env var CONFIG_NETWORK_URL.

Is this PR related with an open issue?

Related to Issue #1032

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• Follows the code style of this project.
• Tests Cover Changes
• Documentation

[calina-c]

I don't understand the idea behind this PR. Aquarius is multichain, why is there one NETWORK CONFIG URL instead of the multichain defined ones? Alternately, why do we need this variable if we have NETWORK URL?

[mariacarmina]

I don't understand the idea behind this PR. Aquarius is multichain, why is there one NETWORK CONFIG URL instead of the multichain defined ones? Alternately, why do we need this variable if we have NETWORK URL?

NETWORK CONFIG URL is an envvar to compare with event monitor chain. NETWORK CONFIG URL can be a public RPC which points to the same network as EVENTS RPC. It was an incident when our production RPCs for mumbai indexed blocks from polygon mainnet & a check for chain ID could solve the problem by stopping Aquarius from indexing blocks.
Regarding NETWORK URL, I thought that it has a different use case or it's used for another scope, so I did not want to interfere.

[calina-c]

I don't understand the idea behind this PR. Aquarius is multichain, why is there one NETWORK CONFIG URL instead of the multichain defined ones? Alternately, why do we need this variable if we have NETWORK URL?

NETWORK CONFIG URL is an envvar to compare with event monitor chain. NETWORK CONFIG URL can be a public RPC which points to the same network as EVENTS RPC. It was an incident when our production RPCs for mumbai indexed block from polygon mainnet & a check for chain ID could solve the problem by stopping Aquarius from indexing blocks. Regarding NETWORK URL, I thought that it has a different use case or it's used for another scope, so I did not want to interfere.

I think you can use the network url.

[codeclimate]

Code Climate has analyzed commit 271ca2d and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 84.4% (0.0% change).

View more on Code Climate.