fix: removed fallback to prod everclear config

Author: otsybizov

packages/agents/relayer/src/config.ts

@@ -1,6 +1,6 @@
 /* eslint-disable @typescript-eslint/no-explicit-any */
 import * as fs from 'fs';
-import { getEverclearConfig, ajv, EVERCLEAR_CONFIG_URL, getDefaultABIConfig } from '@chimera-monorepo/utils';
+import { getEverclearConfig, ajv, getDefaultABIConfig } from '@chimera-monorepo/utils';
 import { RelayerConfig, RelayerConfigSchema } from './lib/entities';
 import { ChainConfig } from './lib/entities';
 
@@ -29,7 +29,7 @@ export const getEnvConfig = async (): Promise<RelayerConfig> => {
   }
 
   const everclearConfigUrl =
-    process.env.EVERCLEAR_CONFIG || configJson.everclearConfig || configFile.everclearConfig || EVERCLEAR_CONFIG_URL;
+    process.env.EVERCLEAR_CONFIG || configJson.everclearConfig || configFile.everclearConfig || undefined;
 
   const everclearConfig = await getEverclearConfig(everclearConfigUrl);
   const everclearChains = everclearConfig?.chains ?? {};

packages/utils/src/helpers/config.ts

@@ -4,8 +4,6 @@ import { axiosGet } from './axios';
 import { Static, Type } from '@sinclair/typebox';
 import { Logger } from '../logging';
 
-export const EVERCLEAR_CONFIG_URL = 'https://raw.githubusercontent.com/connext/chaindata/main/everclear.json';
-
 export const parseEverclearConfig = (data: object): EverclearConfig => {
   const everclearConfig = data as EverclearConfig;
 
@@ -20,21 +18,12 @@ export const parseEverclearConfig = (data: object): EverclearConfig => {
   return everclearConfig;
 };
 
-export const getEverclearConfig = async (_configUrl?: string): Promise<EverclearConfig | undefined> => {
-  const configUrl = _configUrl ?? EVERCLEAR_CONFIG_URL;
-
+export const getEverclearConfig = async (configUrl: string): Promise<EverclearConfig | undefined> => {
   try {
     const res = await axiosGet(configUrl);
-    const everclearConfig = parseEverclearConfig(res.data);
-    return everclearConfig;
+    return parseEverclearConfig(res.data);
   } catch (err: unknown) {
-    try {
-      const res = await axiosGet(EVERCLEAR_CONFIG_URL);
-      if (res.data) return parseEverclearConfig(res.data);
-    } catch (err: unknown) {
-      return undefined;
-    }
-
+    console.error(`Failed to fetch everclear config from ${configUrl}:`, err);
     return undefined;
   }
 };

packages/utils/test/helpers/config.spec.ts

@@ -1,7 +1,7 @@
 import { SinonStub, stub, restore } from 'sinon';
 import Axios from 'axios';
 
-import { EVERCLEAR_CONFIG_URL, expect, getEverclearConfig, mock, parseEverclearConfig } from '../../src';
+import { expect, getEverclearConfig, mock, parseEverclearConfig } from '../../src';
 
 describe('Helpers:Config', () => {
   describe('#parseEverclearConfig', () => {
@@ -31,26 +31,11 @@ describe('Helpers:Config', () => {
       restore();
     });
 
-    it('should return undefined if it axios.get fails and fallback fails', async () => {
+    it('should return undefined if url fails', async () => {
       getMock.resolves({});
       const config = await getEverclearConfig('http://foo.com');
       expect(config).to.be.undefined;
-      expect(getMock.firstCall.firstArg).to.be.eq('http://foo.com');
-      expect(getMock.secondCall.firstArg).to.be.eq(EVERCLEAR_CONFIG_URL);
-    });
-
-    it('should query fallback if url fails', async () => {
-      getMock.onFirstCall().resolves({});
-      const config = await getEverclearConfig('http://foo.com');
-      expect(config).to.be.deep.eq(mock.config());
-      expect(getMock.firstCall.firstArg).to.be.eq('http://foo.com');
-      expect(getMock.secondCall.firstArg).to.be.eq(EVERCLEAR_CONFIG_URL);
-    });
-
-    it('should work with no url', async () => {
-      const config = await getEverclearConfig();
-      expect(config).to.be.deep.eq(mock.config());
-      expect(getMock.calledOnceWith(EVERCLEAR_CONFIG_URL)).to.be.true;
+      expect(getMock.calledOnceWith('http://foo.com')).to.be.true;
     });
 
     it('should work with url', async () => {