fix: ensure account addresses are checksummed

Swivel-Finance · Dec 8, 2023 · 2833871 · 2833871
1 parent da08e88
commit 2833871
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/src/connection/providers/browser/provider.ts b/src/connection/providers/browser/provider.ts
@@ -1,4 +1,4 @@
-import { providers } from 'ethers';
+import { providers, utils } from 'ethers';
 import { ListenerManager, schedule } from '../../../utils/index.js';
 import { ERRORS } from '../../errors.js';
 import { CONNECTION_EVENTS, type Account, type ConnectionEvents, type ConnectionProvider } from '../../interfaces.js';
@@ -101,6 +101,9 @@ export class BrowserConnectionProvider implements ConnectionProvider {
 
         // lookup ens names for connected accounts
         const accounts = await Promise.all<Account>(addresses.map(async (address) => {
+            // checksum the address
+            address = utils.getAddress(address);
+            // lookup the ens name for the address
             const ensAddress = await provider.lookupAddress(address)
                 .catch((error) => {
                     console.warn(`ENS lookup failed for ${ address }: `, error);

diff --git a/src/connection/providers/mock/provider.ts b/src/connection/providers/mock/provider.ts
@@ -1,6 +1,6 @@
 import { Provider } from '@ethersproject/abstract-provider';
 import { Signer } from '@ethersproject/abstract-signer';
-import { BigNumber } from 'ethers';
+import { BigNumber, utils } from 'ethers';
 import { ListenerManager, schedule } from '../../../utils/index.js';
 import { CONNECTION_EVENTS, type ConnectionEvents, type ConnectionProvider } from '../../interfaces.js';
 import { DEFAULT_CONNECTION_OPTIONS, MockConnection, MockConnectionOptions } from './connection.js';
@@ -79,7 +79,8 @@ export class MockConnectionProvider implements ConnectionProvider {
                 name: connectionOptions.networkName,
             },
             account: {
-                address: connectionOptions.address,
+                // ensure address is checksummed
+                address: utils.getAddress(connectionOptions.address),
                 ensAddress: connectionOptions.ensAddress,
             },
             provider,